Ticket #8487 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Can't see all items in the Control Panel in Kreyol

Reported by: kimquirk Owned by: kimquirk
Priority: high Milestone: 8.2.0 (was Update.2)
Component: sugar Version: not specified
Keywords: blocks:8.2.0 r+ Cc: gregorio, andresambrois, bastienguerry@…
Action Needed: test in release Verified: no
Deployments affected: Blocked By:
Blocking:

Description

Build 8.2-760 on a Kreyol keyboard

I went to the control panel and the translated words are long enough that all the items don't fit in the window of the control panel. Specifically you can't see 'translation' item to choose a language.

There is no way to scroll horizontally or vertically to get to that item. I will not write up individual bugs for each language that I find this problem... but we can assume there will be more.

Test case: http://wiki.laptop.org/go/Tests/Keyboard_mappings

Attachments

8487.patch (1.7 kB) - added by andresambrois 6 years ago.
cp_haiti.png (39.1 kB) - added by erikos 6 years ago.
haiti in sugar-jhbuild

Change History

  Changed 6 years ago by cjb

Marco, Kim, propose a milestone for this?

  Changed 6 years ago by kimquirk

  • cc gregorio added
  • keywords blocks?:8.2.0 added
  • owner changed from marco to erikos
  • milestone changed from Not Triaged to 8.2.0 (was Update.2)

Changing ownership for simon to help figure out the work effort. Also added 'blocks?' so we can re-evaluate the blockerness based on what countries are depending on 8.2. Probably this will be moved to 8.2.1.

  Changed 6 years ago by gregorio

  • keywords blocks:8.2.0 added; blocks?:8.2.0 removed

Hi Guys,

Haiti plans to use 8.2 so we must fix this.

Thanks,

Greg S

  Changed 6 years ago by erikos

I already filed one about that issue #8148

Changed 6 years ago by andresambrois

  Changed 6 years ago by andresambrois

  • keywords r? added

This patch adds a gtk.ScrolledWindow to the control panel, it may not be a pretty fix, but it works and its simple.

  Changed 6 years ago by erikos

  • keywords r- added; r? removed

I don't think we want a scrolled window here - since we have enough space. Adding the xoptions=gtk.SHRINK option when attaching items to the table does solve the issues. But I am not sure yet this is the best fix neither.

  Changed 6 years ago by andresambrois

  • cc andresambrois added

Changed 6 years ago by erikos

haiti in sugar-jhbuild

follow-up: ↓ 9   Changed 6 years ago by gregorio

  • cc bastienguerry@… added

Hi Guys,

I wouldn't worry about the aesthetics at this stage. We can revisit this in 8.2.1 if needed.

Which solution has the list risk to damage or cause problems elsewhere?

Which is the easiest to implement and test?

Which ever solution wins on those two counts sounds like the right strategy to get this closed without delaying the release.

Thanks,

Greg S

in reply to: ↑ 8   Changed 6 years ago by marco

There are two possible short time solutions here:

  • Reimplement the icon view using hippo.
  • Add a scrollbar to the view, which will be visible for languages that overflow.

The hippo solution would be good enough until post 9.1 (at least).

Replying to gregorio:

Which solution has the list risk to damage or cause problems elsewhere?

Hippo is only slightly more risky I think (the code changes would be larger but easy to test).

Which is the easiest to implement and test?

Scrollbar should be easier to implement (I can't say for sure until we tried it though). Test effort should be equivalent.

  Changed 6 years ago by cscott

Another option is to punt this to the translators: ask the translators to please provide shorter strings, either by using abbreviations or by choosing an alternate wording.

  Changed 6 years ago by erikos

  • keywords r+ added; r- removed
  • next_action changed from never set to package

Actually we pushed the scrolled window for now (thanks andres) and do better laying out the icons for 0.82.1 (using hippo).

  Changed 6 years ago by gregorio

Sounds good!

Call it done for now.

Greg S

  Changed 6 years ago by marco

  • next_action changed from package to test in build

  Changed 6 years ago by erikos

  • next_action changed from test in build to approve for release

package: sugar-0.82.8-1.olpc3

|TestCase|

Open the control panel and switch to using the Haiti (kreyol). Open the control panel again. There should be a horizontal scrollbar that let's you scroll to all the items

  Changed 6 years ago by mstone

  • next_action changed from approve for release to add to release

Approved; thanks for the clear message!

  Changed 6 years ago by cscott

  • summary changed from Can't see all items in the Control Panel to Can't see all items in the Control Panel in Kreyol

  Changed 6 years ago by cscott

  • next_action changed from add to release to test in release

Please test build 761 resulting from build-repo commit b7c11be9.

  Changed 6 years ago by mchua

  • owner changed from erikos to kimquirk

assigning to kimquirk to test as per kim's request to test i18n bugs for this round.

  Changed 6 years ago by mchua

  • status changed from new to closed
  • resolution set to fixed

Verified in 766, Kreyol keyboard machine in Kim's office. It's not super-pretty to have to scroll so far for this, but you can see/access all options. Closing.

Note: See TracTickets for help on using tickets.