Patch #21257

Use PDF icon for "Also available in PDF"

Added by Daniel Ritz almost 2 years ago. Updated almost 2 years ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:UI
Target version:-

0002-Use-PDF-icon-for-Also-available-in-PDF.patch Magnifier (1002 Bytes) Daniel Ritz, 2015-11-17 15:32


Related issues

Related to Redmine - Patch #14451: Icons for issue and gantt export New
Related to Redmine - Patch #21256: Use CSS instead of image_tag() to show icons for better t... Closed
Duplicated by Redmine - Patch #21245: UI/CSS refinements, part 1 Closed

History

#1 Updated by Go MAEDA almost 2 years ago

  • Duplicated by Patch #21245: UI/CSS refinements, part 1 added

#2 Updated by Go MAEDA almost 2 years ago

  • Related to Patch #14451: Icons for issue and gantt export added

#3 Updated by Go MAEDA almost 2 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 almost 2 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 almost 2 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 almost 2 years ago

  • Related to Patch #21256: Use CSS instead of image_tag() to show icons for better theming support added

Also available in: Atom PDF