[Dash] Previews do not match visual designs
Bug #1224555 reported by
Oren Horev
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu UX |
Fix Released
|
High
|
Xi Zhu | ||
unity8 (Ubuntu) |
Fix Released
|
High
|
Ying-Chun Liu |
Bug Description
dash previews do not match visual designs targeted for 13.10 (across music, videos, apps scopes)
key discrepancies :
1. Header was removed. Exit point is now only through the scope context area at bottom.
2. button should be positioned below Icon and title not above
3. Text description section should appear below button
4. bottom crop line should be positioned at the edge of the scope context area, not above it.
Desired solution
----------------
see visual design spec
Related branches
lp:~unity-team/unity8/fix-genericpreview
- PS Jenkins bot (community): Approve (continuous-integration)
- Michał Sawicz: Approve
- Ying-Chun Liu (community): Approve
- Michal Hruby (community): Approve
-
Diff: 1937 lines (+879/-663)14 files modifiedDash/Apps/AppPreview.qml (+81/-134)
Dash/DashPreview.qml (+97/-99)
Dash/Generic/GenericPreview.qml (+166/-78)
Dash/Movie/MoviePreview.qml (+71/-0)
Dash/PreviewDelegateMapper.qml (+1/-0)
Dash/Previews/Header.qml (+109/-0)
Dash/Video/VideoPreview.qml (+0/-210)
po/unity8.pot (+23/-33)
tests/qmltests/CMakeLists.txt (+2/-1)
tests/qmltests/Dash/Apps/tst_AppPreview.qml (+61/-25)
tests/qmltests/Dash/Movie/tst_MoviePreview.qml (+85/-0)
tests/qmltests/Dash/Video/tst_VideoPreview.qml (+0/-63)
tests/qmltests/Dash/tst_DashPreview.qml (+67/-20)
tests/qmltests/Dash/tst_GenericPreview.qml (+116/-0)
summary: |
- [Dash] Previews shouldn't have headers + [Dash] Previews do not match visual designs |
tags: | added: dash-august-13 |
Changed in unity8: | |
importance: | Undecided → High |
assignee: | nobody → Albert Astals Cid (aacid) |
Changed in unity8: | |
status: | New → Triaged |
milestone: | none → ubuntu-13.10 |
Changed in ubuntu-ux: | |
assignee: | nobody → Xi Zhu (xi.zhu) |
description: | updated |
description: | updated |
Changed in ubuntu-ux: | |
status: | New → Fix Committed |
Changed in ubuntu-ux: | |
importance: | Undecided → High |
Changed in unity8 (Ubuntu): | |
status: | New → Fix Committed |
Changed in unity8: | |
status: | Fix Committed → Fix Released |
tags: | added: has-test |
Changed in ubuntu-ux: | |
status: | Fix Committed → Fix Released |
Changed in unity8 (Ubuntu): | |
assignee: | nobody → Ying-Chun Liu (paulliu) |
importance: | Undecided → High |
no longer affects: | unity8 |
To post a comment you must log in.
I'm working on this bug. So assigned to myself.