Merge lp:~tiagosh/unity-2d/bugfix-893061 into lp:unity-2d

Proposed by Tiago Salem Herrmann
Status: Merged
Approved by: Gerry Boland
Approved revision: 797
Merged at revision: 841
Proposed branch: lp:~tiagosh/unity-2d/bugfix-893061
Merge into: lp:unity-2d
Diff against target: 25 lines (+6/-1)
2 files modified
places/FilterPane.qml (+0/-1)
tests/manual-tests/places.txt (+6/-0)
To merge this branch: bzr merge lp:~tiagosh/unity-2d/bugfix-893061
Reviewer Review Type Date Requested Status
Gerry Boland (community) Approve
Review via email: mp+83431@code.launchpad.net

Commit message

[dash] Remove underline from "Filter results" when it is highlighted

Description of the change

The width of "Filter Results" string is being changed from 99 to 100 everytime it gets underlined. (the blur effect does not change the width though)
So here is a hack to prevent the Text element to be resized whenever the font is changed.

To post a comment you must log in.
Revision history for this message
Gerry Boland (gerboland) wrote :

I think that underline is strange. Could we get Design to confirm that it is correct?

Revision history for this message
Gerry Boland (gerboland) wrote :

Have attached Ayatana Design to bug, waiting on clarification from them.

review: Needs Information
Revision history for this message
Gerry Boland (gerboland) wrote :

The underline is incorrect. Removing it should fix this issue, and there'll be need for this workaround.

review: Needs Fixing
lp:~tiagosh/unity-2d/bugfix-893061 updated
794. By Tiago Salem Herrmann

rebase

795. By Tiago Salem Herrmann

[places] remove underline from "Filter Results" when highlighted

Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

done. rebased and removed the underline.

Revision history for this message
Gerry Boland (gerboland) wrote :

Thanks Tiago!

Can you please do the following:
Add a quick Manual Test to check this in tests/manual-tests/places.txt

lp:~tiagosh/unity-2d/bugfix-893061 updated
796. By Tiago Salem Herrmann

[places] add manual test

797. By Tiago Salem Herrmann

[places] add lp bug number

Revision history for this message
Tiago Salem Herrmann (tiagosh) wrote :

> Thanks Tiago!
>
> Can you please do the following:
> Add a quick Manual Test to check this in tests/manual-tests/places.txt

done! thanks.

Revision history for this message
Gerry Boland (gerboland) wrote :

Perfect, thank you!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'places/FilterPane.qml'
2--- places/FilterPane.qml 2011-12-19 10:44:26 +0000
3+++ places/FilterPane.qml 2011-12-21 15:35:33 +0000
4@@ -69,7 +69,6 @@
5 text: u2d.tr("Filter results")
6 font.bold: true
7 fontSize: "large"
8- font.underline: parent.state == "selected"
9
10 anchors.verticalCenter: parent.verticalCenter
11 anchors.right: arrow.left
12
13=== modified file 'tests/manual-tests/places.txt'
14--- tests/manual-tests/places.txt 2011-12-07 15:01:18 +0000
15+++ tests/manual-tests/places.txt 2011-12-21 15:35:33 +0000
16@@ -11,3 +11,9 @@
17 --> Dash opens and "Run a command" is shown in the search field - (lp:883392)
18
19 ----
20+
21+ * Hit super key to Open Dash, select the file lens and press right arrow key
22+
23+--> The text "Filter results" gets highlighted and no underline is shown. (lp:893061)
24+
25+----

Subscribers

People subscribed via source and target branches