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
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.
lp:~rohangarg/ubiquity/ubiquity updated
5949. By Dimitri John Ledkov

Use pyflakes3 instead of pyflakes, for better python3 checks.

Revision history for this message
Aurélien Gâteau (agateau) wrote :

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.

review: Approve
lp:~rohangarg/ubiquity/ubiquity updated
5950. By Jonathan Riddell

[ Aurélien Gâteau ]
* Merge lp:~agateau/ubiquity/kde-rework-try-install-buttons
 - Make the icon part of the buttons so that they are clickable as
   well (I always click the icon and get frustrated it does not do
   anything...)
 - Use latest Oxygen icons

5951. By Colin Watson

merge lp:~laney/ubiquity/iso-639-b

5952. By Colin Watson

releasing version 2.15.9

Revision history for this message
Dimitri John Ledkov (xnox) wrote :

Should this be merged, or agateau's suggestion used?

review: Needs Information
lp:~rohangarg/ubiquity/ubiquity updated
5953. By Dimitri John Ledkov

Change Ubuntu Studio background in ubiquity-dm.

5954. By Dimitri John Ledkov

* Don't tell automake to install everything to ubiquity-frontend-gtk and
  try to clean up after
* Reorganize app icon handling
  - Fixes missing app icon in GNOME Shell (LP: #1164573)
  - Use improved icon from Humanity as fallback icon
* Update *.install files for above changes
* Fix minor lintian warning by not installing empty /usr/share/applications/

5955. By Jonathan Riddell

[ Aurélien Gâteau ]
KDE: Set icon theme to Oxygen, shows icons on standalone Ubiquity

Revision history for this message
Rohan Garg (rohangarg) wrote :

Both suggestions are optimizations, and it'll be good to have both. I'll add Aurélien suggestion to the code as well.

lp:~rohangarg/ubiquity/ubiquity updated
5956. By Dimitri John Ledkov

Add initial autopilot support! UI testing, brave new world!

5957. By Stéphane Graber

Always install shim-signed, grub-efi-amd64-signed and linux-signed-generic on UEFI systems.

5958. By Stéphane Graber

Update d-i sources.

5959. By Stéphane Graber

Fix mixed tabs/spaces in test_ubiquity_custom.py

5960. By Stéphane Graber

Fix autopkgtests to pass pyflakes.

5961. By Stéphane Graber

releasing version 2.15.10

5962. By Stéphane Graber

Update d-i sources.

5963. By Stéphane Graber

releasing version 2.15.11

5964. By Dimitri John Ledkov

[qt] Fix language drop down list closes on first click (LP:
#1187762)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubiquity/plugins/ubi-language.py'
2--- ubiquity/plugins/ubi-language.py 2013-07-08 14:45:24 +0000
3+++ ubiquity/plugins/ubi-language.py 2013-07-18 14:32:28 +0000
4@@ -398,6 +398,9 @@
5 from PyQt4.QtGui import QWidget, QPixmap, QIcon
6 self.page = uic.loadUi('/usr/share/ubiquity/qt/stepLanguage.ui')
7 self.combobox = self.page.language_combobox
8+ # Tell layout that all items are of uniform sizes
9+ # Fixes LP:1187762
10+ self.combobox.view().setUniformItemSizes(True)
11 self.combobox.currentIndexChanged[str].connect(
12 self.on_language_selection_changed)
13 if not self.controller.oem_config:
14@@ -502,8 +505,7 @@
15 def set_language_choices(self, choices, choice_map):
16 PageBase.set_language_choices(self, choices, choice_map)
17 self.combobox.clear()
18- for choice in choices:
19- self.combobox.addItem(str(choice))
20+ self.combobox.addItems(choices)
21
22 def set_language(self, language):
23 index = self.combobox.findText(str(language))

Subscribers

People subscribed via source and target branches

to status/vote changes: