Patch #21257
openUse PDF icon for "Also available in PDF"
0%
Files
Related issues
Updated by Go MAEDA about 9 years ago
- Has duplicate Patch #21245: UI/CSS refinements, part 1 added
Updated by Go MAEDA about 9 years ago
- Related to Patch #14451: Icons for issue and gantt export added
Updated by Go MAEDA about 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 about 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 about 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 about 9 years ago
- Related to Patch #21256: Use CSS instead of image_tag() to show icons for better theming support added