Merge lp:~aacid/unity-2d/loadhttpicon into lp:unity-2d

Proposed by Albert Astals Cid
Status: Merged
Approved by: Paweł Stołowski
Approved revision: 1040
Merged at revision: 1044
Proposed branch: lp:~aacid/unity-2d/loadhttpicon
Merge into: lp:unity-2d
Diff against target: 33 lines (+14/-1)
2 files modified
shell/dash/TileVertical.qml (+2/-1)
tests/manual-tests/dash.txt (+12/-0)
To merge this branch: bzr merge lp:~aacid/unity-2d/loadhttpicon
Reviewer Review Type Date Requested Status
Paweł Stołowski (community) Approve
Review via email: mp+100755@code.launchpad.net

Commit message

[Dash] Load http:// iconHints directly instead of prepending the image:// part

Description of the change

[Dash] Load http:// iconHints directly instead of prepending the image:// part

To post a comment you must log in.
Revision history for this message
Paweł Stołowski (stolowski) wrote :

Great stuff!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'shell/dash/TileVertical.qml'
2--- shell/dash/TileVertical.qml 2011-12-01 01:41:13 +0000
3+++ shell/dash/TileVertical.qml 2012-04-04 09:11:24 +0000
4@@ -78,7 +78,8 @@
5 Image {
6 id: icon
7
8- source: iconHint != "" ? "image://icons/"+iconHint : "image://icons/unknown"
9+ source: iconHint.indexOf("http://") == 0 ? iconHint
10+ : "image://icons/" + (iconHint != "" ? iconHint : "unknown")
11 onStatusChanged: if (status == Image.Error) source = "image://icons/unknown"
12 width: 64
13 height: 64
14
15=== modified file 'tests/manual-tests/dash.txt'
16--- tests/manual-tests/dash.txt 2012-04-02 11:38:16 +0000
17+++ tests/manual-tests/dash.txt 2012-04-04 09:11:24 +0000
18@@ -113,3 +113,15 @@
19 # * Open dash and verify that all results are displayed in 'Home' lens, and no search query is applied.
20
21 -----
22+
23+ # Test case objectives:
24+ # * Check that remote icons show in the dash
25+ # Pre-conditions
26+ # * None
27+ # Test steps
28+ # * Open dash
29+ # * Go to the music lens
30+ # * Type some song you don't have on the local computer and that is on the music store
31+ # * Verify the results have the cover of the song/album and not the ? icon
32+
33+-----

Subscribers

People subscribed via source and target branches