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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bojhan’s picture

Yhea, and it awkwardly offsets all the titles. I would definitely love to see this fixed.

eliza411’s picture

Issue tags: +D.o UX
mgifford’s picture

We should be able to leverage theme_file_icon() for this, right?

tvn’s picture

Project: [Archive] Drupal.org D7 upgrade QA » Extended File Field
Version: » 7.x-1.x-dev
jthorson’s picture

Currently, this is default theme_file_link() output ... is there a better theme function you'd like to see?

jthorson’s picture

Priority: Normal » Minor

Lowering 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.

drumm’s picture

Project: Extended File Field » Drupal.org customizations
Version: 7.x-1.x-dev » 7.x-3.x-dev

This 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.)

sun’s picture

Am I mistaken, or isn't this configurable via the Field UI even?

joachim’s picture

I 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.

yoroy’s picture

Priority: Minor » Normal
FileSize
141.22 KB

Looks 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:

mockup of issue queue file list with and without file type icons

yoroy’s picture

Oops, new to dreditor, those buttons normally aren't there of course. Updated screenshot:

mockup without dreditor buttons

drumm’s picture

Issue tags: +drupal.org hitlist
drumm’s picture

isn't this configurable via the Field UI even?

Not that I know of. theme_file_link() hard-codes calling theme('file_icon', …). We can override theme_file_icon() to return an empty string.

drumm’s picture

Assigned: Unassigned » drumm
drumm’s picture

Issue summary: View changes
Status: Active » Needs review
drumm’s picture

Status: Needs review » Fixed
Issue tags: +needs drupal.org deployment

  • drumm committed e27aac7 on 7.x-3.x
    Issue #2128907: Remove fileicons from file upload views
    
drumm’s picture

This is deployed now. File icons on comments will remain in places as caches, especially the comment render cache, clear.

YesCT’s picture

nice. thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

drumm’s picture

Issue tags: -needs drupal.org deployment