Patch #21257
Use PDF icon for "Also available in PDF"
Status: | New | Start date: | ||
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | UI | |||
Target version: | - |
Related issues
History
#1
Updated by Go MAEDA over 6 years ago
- Duplicated by Patch #21245: UI/CSS refinements, part 1 added
#2
Updated by Go MAEDA over 6 years ago
- Related to Patch #14451: Icons for issue and gantt export added
#3
Updated by Go MAEDA over 6 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.
#4
Updated by budo kaiman over 6 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.
#5
Updated by Daniel Ritz over 6 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...
#6
Updated by Toshi MARUYAMA over 6 years ago
- Related to Patch #21256: Use CSS instead of image_tag() to show icons for better theming support added