source file listing shows icons but does not render them well
Bug #1203583 reported by
Nick Veitch
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-gui |
Fix Released
|
Low
|
Unassigned |
Bug Description
In the new source view, if you select the icon file, it just gets listed. Actually, I always get a load of lines, sometimes with bits of symbols on them. It would be nice to recognise the .svg type and display the icon I guess.
Related branches
lp:~gary/juju-gui/bug1203583
- Juju GUI Hackers: Pending requested
-
Diff: 155 lines (+98/-2)5 files modifiedapp/models/charm.js (+40/-1)
app/subapps/browser/views/bundle.js (+4/-1)
app/subapps/browser/views/charm.js (+7/-0)
test/test_browser_charm_details.js (+21/-0)
test/test_model.js (+26/-0)
Changed in juju-gui: | |
importance: | Medium → High |
summary: |
- source view of icon files is weird + source file listing is unnecessarily difficult to use |
Changed in juju-gui: | |
status: | Triaged → Fix Committed |
Changed in juju-gui: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Thanks Nick. It took me a minute to parse this as we just recently changed the tab name to "Source".
We can either try to detect the content-type and adjust how we display the content. The other way to go is to remove non-text content from the list of files we show in the drop down. I'm not sure which way people think would make the most sense.