-
-
Notifications
You must be signed in to change notification settings - Fork 80
Fix doc icon for temporary v0.9 manual crates list #224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
<a href="/gtk-rs-core/stable/latest/docs/cairo">🕮</a> | ||
<a href="/gtk-rs-core/stable/latest/docs/cairo"> | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
<svg width="181" height="105" version="1.1" viewBox="0 0 47.89 27.781" xmlns="http://www.w3.org/2000/svg" xmlns:cc="http://creativecommons.org/ns#" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of redeclaring the image multiple times, couldn't it be stored in a file instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Even if not redeclared, it's very hard to read anything, it's supposed to be a markdown file)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a temporary (!!!) fix because we are still lacking the v0.9 docs for gtk3 in the new system (cc @bilelmoussaoui). The correct solution would just be {% include crates.html %}
which generates it from a json file. But that would lead to incorrect docs links for now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah i always forget about those. Let me fix them :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you mean by that that we will generate the svg images directly into the HTML, please don't and create an image file instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then you have to fix it yourself. Coloring with filters seems like a complicated detour. I think there is also a way with iframes? But I never used it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could also generate all svgs in the page with the required colors of course.
@@ -36,7 +36,7 @@ The following table contains the most popular crates of **gtk-rs**. More informa | |||
{% include crates.html %} | |||
--> | |||
|
|||
<table> | |||
<table class="crates"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this indent.
Original fix is in #217, but was not applied to the temporary manual crates list.
Closes #223