CP: Switch to GNOME Button not displayed correctly in all the languages

[erikos]

Reproduce:

- change your language to be French

- go to CP and try to switch to GNOME

[erikos]

Screenshot of the issue

[godiard]

Attach a solution proposal.

[godiard]

The first patch used a fixed size of 2/3 of the window instead of 1/2 of the window to the text.

This is not a good solution.

Talked with erikos and now I added a scrolled window. The second patch replace the first.

[dsd]

have you tested it with a variety of languages?

[erikos]

Replying to dsd:

have you tested it with a variety of languages?

Ok, I tested some languages and so far only French seem to have the issue. The simplest fix for 10.1.3 would be to just do:

self._gnome_opt_label.set_size_request(gtk.gdk.screen_width(), -1)

instead of: self._gnome_opt_label.set_size_request(gtk.gdk.screen_width() / 2, -1)

And for the next build we do the scrolling window. But I am ok with moving it out, if others are not convinced it is worth it.

[dsd]

I think it's been decided that it's too late for 10.1.3 (which I agree with).

still want the fix though! patch looks good but my question is whether it has been tested. in addition to fixing french, it should be tested to ensure that there is no change for English, Spanish, and another 2 languages of your testing choice.

[godiard]

Replying to erikos:

Replying to dsd:

have you tested it with a variety of languages?

Ok, I tested some languages and so far only French seem to have the issue. The simplest fix for 10.1.3 would be to just do: self._gnome_opt_label.set_size_request(gtk.gdk.screen_width(), -1) instead of: self._gnome_opt_label.set_size_request(gtk.gdk.screen_width() / 2, -1) And for the next build we do the scrolling window. But I am ok with moving it out, if others are not convinced it is worth it.

I have tested it but does not work (see http://dev.laptop.org/attachment/ticket/10544/text_with_window_width%20spanish%20ERROR.png)

gtk.gdk.screen_width() return the screen width in pixels, not the popup width.

I tried again width 2 / 3 of screen . Is a magic number but 1 / 2 of screen too.

I have tested it in Spanish, French, English, Arabic and Portuguese.

0001-Increase-width-requested-to-the-labels-V2.patch add a variable width to change it in one place.

[godiard]

I have attached screenshots with the scrolled window patch applied in Arabic, English, French, Portuguese and Spanish.

[dsd]

So which patch are you proposing? And did you see any errors in the languages tested? It sounds like using gtk.gdk.screen_width() is wrong, but it's not clear if you are proposing a solution that avoids that.

[godiard]

Daniel, I think the two solutions work, but i don't know what is the best solution at this moment.

I have talked with erikos and he say 2/3 of the screen is a magic number, and i agree, but I don't know if you want a change more invasive like adding a ScrolledWindow now.

[dsd]

No problem with invasive changes, lets fix this properly. I think we already decided that this is not 10.1.3 material.

[godiard]

Replying to dsd:

No problem with invasive changes, lets fix this properly. I think we already decided that this is not 10.1.3 material.

I don't think so. erikos said "The simplest fix for 10.1.3 would be..."

I think we need a fix for 10.1.3

[erikos]

Let's move this to the next milestone.

[martin.langhoff]

Is there no low-risk way to workaround or fix this issue? Can we ask Sam to take a patched version and see if anything breaks...?

[dsd]

I don't think we've identified any "low-risk" fixes right now - as this bug shows just how sensitive the layout is.

Given that this is a minor bug, has a workaround, is not a regression, does not affect any large deployments, and that 10.1.3 is already overdue, I think it should be pushed out.

[godiard]

Replying to martin.langhoff:

Is there no low-risk way to workaround or fix this issue? Can we ask Sam to take a patched version and see if anything breaks...?

Low-risk solution to check by Sam: copy the view.py file in /usr/share/sugar/extensions/cpsection/switchdesktop/

Anyway, dsd will not include any solution in 10.1.3

[erikos]

Let's make sure we get the right fix into the next F14 build, we might get new dance material then: http://www.dailymotion.com/video/xftdx1_clip-nosy-komba_tech

[dsd]

Yeah. What I'd like to see next is clarity on a patch being proposed, and written indication that testing has been performed with no problems found. Gonzalo pointed out on IRC that the existing layout code isn't very good and could do with an overhaul - I'd welcome an invasive patch to fix this properly.

[godiard]

Daniel:

I propose a new patch. This time i removed all the size request in the labels, and justified the texts.

I have added a ScrolledWindow for the cases when the text is longer than the place you can provide in the window.

The justification does not work very well with labels with line wrap, but I tried many different combination and that is the better.

I have tested it with many languages and found no problems.

[dsd]

Thanks, looking good.

label_space looks unused though.

Also, the justification has not been fully maintained. "Sugar is the" is no longer aligned with "You can return" on the main screen, and after choosing to switch, "Restart your computer to" is not aligned with "Remember, you can return"

[godiard]

Replying to dsd:

Thanks, looking good. label_space looks unused though.

It's true. My mistake.

Also, the justification has not been fully maintained. "Sugar is the" is no longer aligned with "You can return" on the main screen, and after choosing to switch, "Restart your computer to" is not aligned with "Remember, you can return"

The justification is in the same paragraph. The paragraphs don't have the same size, because the size is calculated by gtk.

Like the documentation says: "in reality fill, expansion, requested sizes, widget expansion semantics, container packing semantics, electron spins and lunar cycles are computed to determine how much space each widget wins. " http://faq.pygtk.org/index.py?req=show&file=faq12.002.htp

[dsd]

Is it possible to maintain the appearance through some other approach? In my opinion the new version looks jumbled because of this.

[godiard]

Replying to dsd:

Is it possible to maintain the appearance through some other approach? In my opinion the new version looks jumbled because of this.

No really.

If you prefer your approach, may be is better use only a ScrolledWindow and your size request in the labels. I don't like it, but you are the maintainer. I have spent to much time with this.

[dsd]

Simon, what are your thoughts?

[erikos]

I think Gonzalo's approach is ok. Another solution would be to instead of requesting a specific size of the label setting the border of the box to something that gives us nice spacing (style.GRID_CELL_SIZE) or similar.

[dsd]

GTK+-3 seems to do a better job here, probably due to its height-for-width rendering - according to a separate test we can probably just set all widgets to have the width of the largest one.

For GTK+-2 that doesn't quite work, but telling all widgets to use the full space allocated to the container does. pushed a fix in olpc-switch-desktop-0.8 based on gonzalo's work, thanks!

[dsd]

test in 11.2.0 build 19

[greenfeld]

Tested the two screens in the "Switch to Gnome" dialog in Arabic, English, French, Portuguese, Spanish, and a few other languages in 11.2.0 os19.

I did not see any obvious button size or text layout issues, although I am not an expert on International text layouts.