Merge lp:~rohangarg/ubiquity/ubiquity into lp:ubiquity
Proposed by
Rohan Garg
Status: | Merged |
---|---|
Merged at revision: | 5964 |
Proposed branch: | lp:~rohangarg/ubiquity/ubiquity |
Merge into: | lp:ubiquity |
Diff against target: |
23 lines (+4/-2) 1 file modified
ubiquity/plugins/ubi-language.py (+4/-2) |
To merge this branch: | bzr merge lp:~rohangarg/ubiquity/ubiquity |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dimitri John Ledkov | Needs Information | ||
Aurélien Gâteau (community) | Approve | ||
Review via email: mp+172045@code.launchpad.net |
Description of the change
Inform the listview that it can assume all its items to be of uniform size as this greatly reduces the amount of work the UI has to do to layout it's popup. This also workarounds a bug where the combobox had to be clicked twice to get the list to show up
To post a comment you must log in.
Looks good to me. I would just suggest adding a comment before the added line to explain why this is done.
Another possible optimization would be to change set_language_ choices( ) from:
for choice in choices:
self.combobox. addItem( str(choice) )
to:
self. combobox. addItems( choices)
Which is faster, and actually also avoids the problem.