Patch #21257
closed
Use PDF icon for "Also available in PDF"
Added by Daniel Ritz over 10 years ago. Updated 10 months ago.
Files
| 0002-Use-PDF-icon-for-Also-available-in-PDF.patch (1002 Bytes) 0002-Use-PDF-icon-for-Also-available-in-PDF.patch | Daniel Ritz, 2015-11-17 15:32 |
Related issues
Updated by Go MAEDA over 10 years ago
- Has duplicate Patch #21245: UI/CSS refinements, part 1 added
Updated by Go MAEDA over 10 years ago
- Related to Patch #14451: Icons for issue and gantt export added
Updated by Go MAEDA over 10 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 10 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 10 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 10 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 11 months ago
- Related to Feature #42598: Drop Atom icon in "Also available in" links added