Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The fileicons offer little to no functional benefit and just add unnecessary screen noise. They should be removed.
Deployment
Also update project_issue_file_test to 7.x-2.x
, currently 4f5a237
Comment | File | Size | Author |
---|---|---|---|
#11 | 1997692-remove-file-icons-2.jpg | 144.12 KB | yoroy |
#10 | 1997692-remove-file-icons.jpg | 141.22 KB | yoroy |
fileicons.png | 62.5 KB | alexpott |
Comments
Comment #1
Bojhan CreditAttribution: Bojhan commentedYhea, and it awkwardly offsets all the titles. I would definitely love to see this fixed.
Comment #2
eliza411 CreditAttribution: eliza411 commentedComment #3
mgiffordWe should be able to leverage theme_file_icon() for this, right?
Comment #4
tvn CreditAttribution: tvn commentedComment #5
jthorson CreditAttribution: jthorson commentedCurrently, this is default theme_file_link() output ... is there a better theme function you'd like to see?
Comment #6
jthorson CreditAttribution: jthorson commentedLowering priority, due to the effort versus reward equation here ... currently, this is a default theme function, and forcing us to a custom theme function to remove it seems like unnecessary complexity.
Comment #7
drummThis may be done in drupal.org customizations, instead of piling onto extended file field. (We generally want to keep PHP customizations in modules, not the theme.)
Comment #8
sunAm I mistaken, or isn't this configurable via the Field UI even?
Comment #9
joachim CreditAttribution: joachim commentedI find that the file icons are a nice visual cue that tells you what the link is. Without that icon, you only know that the link is a file download from the 'File' header of the table, which is easily missed, and the file extension in the link itself, which requires you to scan the filename and break it down into pieces.
The type of the icon is also an easy way of spotting patches vs screenshots.
Comment #10
yoroy CreditAttribution: yoroy commentedLooks like a good idea to remove them. Mostly clutter indeed and without them we get nicer alignments.
The number of buttons next to each item is a good indicator for wether something is a patch or not, here's a mockup:
Comment #11
yoroy CreditAttribution: yoroy commentedOops, new to dreditor, those buttons normally aren't there of course. Updated screenshot:
Comment #12
drummComment #13
drummNot that I know of.
theme_file_link()
hard-codes callingtheme('file_icon', …)
. We can overridetheme_file_icon()
to return an empty string.Comment #14
drummComment #15
drummThis is running at https://lax1-drupal.redesign.devdrupal.org/node/2469929
Comment #16
drummComment #18
drummThis is deployed now. File icons on comments will remain in places as caches, especially the comment render cache, clear.
Comment #19
YesCT CreditAttribution: YesCT commentednice. thanks.
Comment #21
drumm