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 |
Related bugs: |
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.
PASSED: Continuous integration, rev:243 /core-apps- jenkins. ubuntu. com/job/ calculator- app-ci/ 29/ /core-apps- jenkins. ubuntu. com/job/ generic- update- mp/189/ console
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild: /core-apps- jenkins. ubuntu. com/job/ calculator- app-ci/ 29/rebuild
https:/