Patch #21257
closed
Use PDF icon for "Also available in PDF"
0%
Files
Related issues
Updated by Go MAEDA over 9 years ago
- Has duplicate Patch #21245: UI/CSS refinements, part 1 added
Updated by Go MAEDA over 9 years ago
- Related to Patch #14451: Icons for issue and gantt export added
Updated by Go MAEDA over 9 years ago
Similar patch #14451 was not adopted.
Jean-Phillipe Lang said on #14451#note-2:
I'm not a big fan of icons everywhere. We already have icons for the main actions links (Edit, Watch...) and I'd prefer to keep export links as they are.
Updated by budo kaiman over 9 years ago
Go MAEDA wrote:
Similar patch #14451 was not adopted.
Jean-Phillipe Lang said on #14451#note-2:
I'm not a big fan of icons everywhere. We already have icons for the main actions links (Edit, Watch...) and I'd prefer to keep export links as they are.
The "Also available in" just below uses rss icon for atom, so it would make sense to make this consistent by either adding the pdf icon or removing the rss icon.
Updated by Daniel Ritz over 9 years ago
Go MAEDA wrote:
Similar patch #14451 was not adopted.
Thanks, didn't see that one.
Jean-Phillipe Lang said on #14451#note-2:
I'm not a big fan of icons everywhere. We already have icons for the main actions links (Edit, Watch...) and I'd prefer to keep export links as they are.
I actually kinda agree. Part of the reason I did Patch #21256 is to be able replace all icons with the less prominent FontAwesome icons. Because they're just in the color of the text, they don't stick out as much...
Updated by Toshi MARUYAMA over 9 years ago
- Related to Patch #21256: Use CSS instead of image_tag() to show icons for better theming support added
Updated by Marius BĂLTEANU 20 days ago
- Related to Feature #42598: Drop Atom icon in "Also available in" links added