Merge lp:~shlomister/ubiquity/fix-bug-560114 into lp:ubiquity

Proposed by Shlomil
Status: Rejected
Rejected by: Stéphane Graber
Proposed branch: lp:~shlomister/ubiquity/fix-bug-560114
Merge into: lp:ubiquity
Diff against target: 13 lines (+2/-1)
1 file modified
ubiquity/frontend/gtk_ui.py (+2/-1)
To merge this branch: bzr merge lp:~shlomister/ubiquity/fix-bug-560114
Reviewer Review Type Date Requested Status
Ubuntu Installer Team Pending
Review via email: mp+23648@code.launchpad.net

Commit message

Fix truncated text in Hebrew and RTL languages

Description of the change

This changes makes Hebrew and RTL text readable and not truncated as it is right now.
Unlike the previous branch I submitted, in this branch the lines are all aligned correctly and it looks real good.

Seems like the line I disabled only for RTL causes a lot of trouble for RTL text labels. I don't know whether this is the proper solution because I don't understand what is the purpose of wrap_fix(). Seems to me like everything works OK without it and in Hebrew it works even better without it.

Please do your best to merge this change or fix it in another way before the release. It is extremely important.

Related to bug 560114.

To post a comment you must log in.
Revision history for this message
Stéphane Graber (stgraber) wrote :

Marking as rejected as apparently this patch isn't relevant with the switch to pygi in ubiquity, based on comments and bug status of bug 560114.

Unmerged revisions

4096. By Shlomil

don't mess up allocated width in RTL mode

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubiquity/frontend/gtk_ui.py'
2--- ubiquity/frontend/gtk_ui.py 2010-04-16 10:29:55 +0000
3+++ ubiquity/frontend/gtk_ui.py 2010-04-19 04:39:16 +0000
4@@ -82,7 +82,8 @@
5 old_width, old_height = layout.get_size()
6 if old_width / pango.SCALE == allocation.width:
7 return
8- layout.set_width(allocation.width * pango.SCALE)
9+ if gtk.widget_get_default_direction() != gtk.TEXT_DIR_RTL:
10+ layout.set_width(allocation.width * pango.SCALE)
11 unused, height = layout.get_size()
12 if allocation.height != height / pango.SCALE:
13 w.set_size_request(-1, height / pango.SCALE)

Subscribers

People subscribed via source and target branches

to status/vote changes: