Page MenuHomePhabricator

filepreview_widget: Show "Resolution" field when appropriate.
ClosedPublic

Authored by netstar on Dec 3 2019, 3:25 AM.

Details

Summary

This adds additional checks to ensure we dont add this field
when it shouldn't be there. Currently files like .tgz are
listing a "Resolution" on mouseover in EFM.

Test Plan

Mouse over different file types. See inappropriate field.
Apply patch.
Mouse over file types. Resolution should only show for media.

Diff Detail

Repository
rE core/enlightenment
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
netstar created this revision.Dec 3 2019, 3:25 AM
netstar requested review of this revision.Dec 3 2019, 3:25 AM
netstar updated this revision to Diff 27253.Dec 3 2019, 3:27 AM

Indentation fix.

raster accepted this revision.Jan 1 2020, 5:44 AM
This revision is now accepted and ready to land.Jan 1 2020, 5:44 AM
This revision was automatically updated to reflect the committed changes.