Merge lp:~mzanetti/ubuntu-calculator-app/fix-icon-in-layout into lp:ubuntu-calculator-app

Proposed by Michael Zanetti
Status: Merged
Approved by: Bartosz Kosiorek
Approved revision: 243
Merged at revision: 243
Proposed branch: lp:~mzanetti/ubuntu-calculator-app/fix-icon-in-layout
Merge into: lp:ubuntu-calculator-app
Diff against target: 13 lines (+2/-2)
1 file modified
app/ui/FavouritePage.qml (+2/-2)
To merge this branch: bzr merge lp:~mzanetti/ubuntu-calculator-app/fix-icon-in-layout
Reviewer Review Type Date Requested Status
Bartosz Kosiorek Approve
Jenkins Bot continuous-integration Approve
Review via email: mp+278848@code.launchpad.net

Commit message

fix Icon in layout

The Icon seems a bit picky when it comes to sizing. RowLayout
doesn't care about the size of items, it overwrites them.
The combination of the two a) produces warnings and b) makes
the app freeze sometimes (for example on my desktop).

To post a comment you must log in.
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Bartosz Kosiorek (gang65) wrote :

Work ok for me

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'app/ui/FavouritePage.qml'
2--- app/ui/FavouritePage.qml 2015-11-09 14:18:18 +0000
3+++ app/ui/FavouritePage.qml 2015-11-27 16:42:43 +0000
4@@ -76,8 +76,8 @@
5 anchors.horizontalCenter: parent.horizontalCenter
6 Icon {
7 id: favouriteIcon
8- height: parent.height - units.gu(2)
9- width: height
10+ Layout.preferredHeight: parent.height - units.gu(2)
11+ Layout.preferredWidth: height
12
13 Layout.alignment: Qt.AlignVCenter
14

Subscribers

People subscribed via source and target branches