Bug 20360 - "Done" button off-screen when custom partitioning, the partitioning screen also overlaps the left panel.
Summary: "Done" button off-screen when custom partitioning, the partitioning screen al...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: Installer (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: release_blocker normal
Target Milestone: Mageia 6
Assignee: Mageia tools maintainers
QA Contact:
URL:
Whiteboard:
Keywords: PATCH
: 20361 20438 20575 (view as bug list)
Depends on:
Blocks: 20624
  Show dependency treegraph
 
Reported: 2017-02-26 20:24 CET by psyca
Modified: 2017-04-04 12:02 CEST (History)
9 users (show)

See Also:
Source RPM: drakx-installer-stage2
CVE:
Status comment:


Attachments
Screenshot from Netinstall (38.65 KB, image/png)
2017-02-26 20:25 CET, psyca
Details
Aurélien's report.bug.xz (209.56 KB, application/x-xz)
2017-02-27 11:25 CET, Aurélien Murith
Details
Screenshot with button text translated (64.95 KB, image/png)
2017-03-15 01:23 CET, Martin Whitaker
Details
Screenshot without button text translated (78.39 KB, image/png)
2017-03-15 01:25 CET, Martin Whitaker
Details
do not make all action buttons the same size (1.12 KB, patch)
2017-03-15 07:52 CET, Thierry Vignaud
Details | Diff
shrink "expert/normal mode" string (47.39 KB, patch)
2017-03-15 07:52 CET, Thierry Vignaud
Details | Diff
english partitioning screen (89.36 KB, image/jpeg)
2017-03-15 23:57 CET, psyca
Details

Description psyca 2017-02-26 20:24:35 CET
Description of problem:

The Installer/Partitioningscreen for the "custom partitioning" doesnt fit on the screen when german language is selected.

See Screenshot
Comment 1 psyca 2017-02-26 20:25:22 CET
Created attachment 8993 [details]
Screenshot from Netinstall

Screenshot from Netinstall 26. Feb. 2017
Comment 2 Marja Van Waes 2017-02-27 07:11:32 CET
I really thought there was a recent bug report for a half-visible "done"-button in this screen, and equally badly visible "next" etc. buttons in consequent screens, but I don't manage to find an open or closed recent report.

(A very similar issue did get fixed in the past, though)

You mention you got this error when doing an install in German. Is that because you don't see this problem when you're doing a default English install on the same hardware?

Do you mind attaching /root/drakx/report.bug.xz from the install that gave this problem? 

Or, if you didn't manage to get past that screen, please:
* reproduce the issue
* switch to tty2 by pressing "Ctrl + Alt + F2"
* insert a USB-key
* then type "bug", so that report.bug will be written to the key
* attach report.bug

Thx :-)

Keywords: (none) => NEEDINFO
CC: (none) => marja11
Assignee: bugsquad => mageiatools
Summary: Displayerror DrakX-Installer - custom partitioning (german) => "Done" button off-screen when custom partitioning (german), the partitioning screen also overlaps the left panel.
Source RPM: (none) => drakx-installer-stage2

Comment 3 Thierry Vignaud 2017-02-27 07:17:38 CET
I doubt that's related to the translations.
It usually happen b/c of the way we size the partition buttons and the particular partition layout we encounter.

CC: (none) => thierry.vignaud

Comment 4 Marja Van Waes 2017-02-27 07:52:40 CET
*** Bug 20361 has been marked as a duplicate of this bug. ***

CC: (none) => aurelienmurith

Comment 5 Aurélien Murith 2017-02-27 11:25:09 CET
Created attachment 8996 [details]
Aurélien's report.bug.xz
Comment 6 psyca 2017-02-28 11:15:19 CET
Tested on an other system again. Its not only in the "custom partitioning" Screen like written in comment 1. It happens also on some systems before (like if you have 2 or more existing partitions).
Comment 7 Samuel Verschelde 2017-02-28 11:22:09 CET
I got the same in VirtualBox not only for the partitioner but also for most other screens.
Samuel Verschelde 2017-02-28 11:22:17 CET

Priority: Normal => High
Target Milestone: --- => Mageia 6

Comment 8 psyca 2017-03-05 23:54:47 CET
Do we still need "NEEDINFO" ?
Rémi Verschelde 2017-03-05 23:58:27 CET

Keywords: NEEDINFO => (none)

Comment 9 Thierry Vignaud 2017-03-10 10:07:51 CET
*** Bug 20438 has been marked as a duplicate of this bug. ***

CC: (none) => yleny

Comment 10 Thierry Vignaud 2017-03-10 10:20:28 CET
A new variation of bug#12422, bug#11988, bug#14839, bug#15272, bug#15264 ... :-(

CC: (none) => mageia

Comment 11 Y.LE_NY 2017-03-10 14:36:57 CET
Hello,

For my ticket : mageia 6 sta2 installer : the window is too big, can not see the right side with buttons ( https://bugs.mageia.org/show_bug.cgi?id=20438)

I launch a new install with the mageia 6 sta 2 installer to help you to fix the problem.

In the mageia installer, the window start to be too big when I choose custom partitioning.
Then in the console log, juste after the window is too big, I have the following error message :

Entering step 'Partitioning'

(runinstall2:730): Gtk-WARNING **: Theme parsing error: diskdrake.css:86:25: The style property GtlPaned:handle-size is deprecated and shouldn't be used anymore.it will removed in a future version. GLib-CRITICAL **: Source ID 687 was not found when attempting to remove it at /usr/lib/libDrakX/interactive/gtk.pm line 924. 

(runinstall2:730): Gtk-WARNING **: Theme parsing error: diskdrake.css:86:25: The style property GtlPaned:handle-size is deprecated and shouldn't be used anymore.it will removed in a future version.

(runinstall2:730): Gtk-WARNING **: Theme parsing error: diskdrake.css:86:25: The style property GtlPaned:handle-size is deprecated and shouldn't be used anymore.it will removed in a future version.
Comment 12 Y.LE_NY 2017-03-10 14:55:45 CET
When (In reply to Y.LE_NY from comment #11)
> Hello,
> 
> For my ticket : mageia 6 sta2 installer : the window is too big, can not see
> the right side with buttons ( https://bugs.mageia.org/show_bug.cgi?id=20438)
> 
> I launch a new install with the mageia 6 sta 2 installer to help you to fix
> the problem.
> 
> In the mageia installer, the window start to be too big when I choose custom
> partitioning.
> Then in the console log, juste after the window is too big, I have the
> following error message :
> 
> Entering step 'Partitioning'
> 
> (runinstall2:730): Gtk-WARNING **: Theme parsing error: diskdrake.css:86:25:
> The style property GtlPaned:handle-size is deprecated and shouldn't be used
> anymore.it will removed in a future version. GLib-CRITICAL **: Source ID 687
> was not found when attempting to remove it at
> /usr/lib/libDrakX/interactive/gtk.pm line 924. 
> 
> (runinstall2:730): Gtk-WARNING **: Theme parsing error: diskdrake.css:86:25:
> The style property GtlPaned:handle-size is deprecated and shouldn't be used
> anymore.it will removed in a future version.
> 
> (runinstall2:730): Gtk-WARNING **: Theme parsing error: diskdrake.css:86:25:
> The style property GtlPaned:handle-size is deprecated and shouldn't be used
> anymore.it will removed in a future version.

When I go in the next steps in the installation process, I have a lot of errors with 
GLib-CRITICAL **: Source ID xxx was not found when attempting to remove it at /usr/lib/libDrakX/interactive/gtk.pm line 924. 

Where the xxx ID source number have an other value.
For example in the parts : Entering step 'Formatting', Entering step 'Choosing packages', Entering step 'Installing', Entering step 'Bootloader'
Marja Van Waes 2017-03-10 15:11:34 CET

Summary: "Done" button off-screen when custom partitioning (german), the partitioning screen also overlaps the left panel. => "Done" button off-screen when custom partitioning, the partitioning screen also overlaps the left panel.

Comment 13 Rémi Verschelde 2017-03-13 16:23:16 CET
Upgrading as release_blocker, as it's a pretty bad looking issue, even if it's not critical. If we release Mageia 6 ~2 years after the previous stable with such a clear visual regression, that wouldn't be good for reviews :/

Priority: High => release_blocker

Comment 14 Martin Whitaker 2017-03-15 01:23:43 CET
Created attachment 9094 [details]
Screenshot with button text translated

In this case the translation is the problem - it's the width of the three buttons below the panel that's forcing the width of the dialogue box to expand. Here's a screenshot with the buttons translated...
Comment 15 Martin Whitaker 2017-03-15 01:25:00 CET
Created attachment 9095 [details]
Screenshot without button text translated

and here's the same thing with the translation disabled just for those buttons.
Comment 16 psyca 2017-03-15 01:29:32 CET
Maybe we/i can change it from "In den Experten-Modus wechseln" (Toggle to expert mode) to "Experten Modus" (expert mode) only ?
Comment 17 psyca 2017-03-15 01:41:31 CET
We have then (maybe) also to modify "Toggle to normal mode" (it apears after clicking on expert mode)...
"Zum Experten-Modus" ; "Zum Normalen-Modus"
Comment 18 Thierry Vignaud 2017-03-15 06:34:04 CET
Indeed.
We may have to check for more languages (at least one of the dup is for French)

It's a side effect of putting all buttons on the same bar:
http://gitweb.mageia.org/software/drakx/commit/perl-install/diskdrake/hd_gtk.pm?id=947ab9bb0fb06611b5bd54465f98545ff175afb2

Compare with mga5 that had 2 button lines:
https://doc.mageia.org/installer/5/en/content/diskdrake.html

One solution would be not to make all buttons equally sized
aka calling set_child_non_homogeneous(0)
Which with mygtk3 style, means replacing:
$per_kind_action_box = gtknew('HButtonBox', layout => 'edge'))
by:
$per_kind_action_box = gtknew('HButtonBox', layout => 'edge', homogenous => 0))
Comment 19 Thierry Vignaud 2017-03-15 06:35:10 CET
And maybe keep the existing "toggle to foobar mode" as a tooltip.
I'll script the translation fixup
Comment 20 Thierry Vignaud 2017-03-15 07:52:29 CET
Created attachment 9096 [details]
do not make all action buttons the same size
Comment 21 Thierry Vignaud 2017-03-15 07:52:33 CET
Created attachment 9097 [details]
shrink "expert/normal mode" string
Comment 22 Marja Van Waes 2017-03-15 10:16:04 CET
Thanks, Thierry :-)

We should ask i18n, to make sure the grammatical case doesn't change when the string is shortened.

Like for Czech
https://bugs.mageia.org/attachment.cgi?id=9097&action=diff#a/perl-install/share/po/cs.po_sec1
I have the feeling "režimu" is incorrect, but that it should be nominative "režim" now.

CC'ing Tomás Kindl and i18n team

Also, in the original Danish translation the word "mode" was ommitted
  
The odd "Ekspert -> Normal" now changed to "Normal" without "->"

And same for:

   msgstr "Normal -> Ekspert"  msgstr "Ekspert"

https://bugs.mageia.org/attachment.cgi?id=9097&action=diff#a/perl-install/share/po/da.po_sec1

CC'ing René Neukirch

CC: (none) => i18n-bugs, rene, supp

Comment 23 psyca 2017-03-15 23:57:20 CET
Created attachment 9101 [details]
english partitioning screen

I have got something similar also in the normal partitioning screen (see english screenshot).
I think its because Windows marked the first partition as "System-reserviert" (system reserved).
As you can see in the screenshot, the filesystem description "Empty" leaves the screen (Emp). Maybe the whole Harddrive partition bar is also to long (?).
Comment 24 psyca 2017-03-16 00:10:25 CET
Please forget comment 23. I can scroll here to the right.
Comment 25 Marja Van Waes 2017-03-16 12:19:06 CET
(In reply to Thierry Vignaud from comment #21)
> Created attachment 9097 [details]
> shrink "expert/normal mode" string


https://ml.mageia.org/l/arc/i18n-discuss/2017-03/msg00023.html

Tarakbumba informed us that: 

> For Turkish they should be: 
> Normal kip
> Uzman kipi


(Btw, what does the option to "Edit Attachment As Comment" do in our Bugzilla? Paste the entire attachment into a comment, with the edits, or...?)
Comment 26 Thierry Vignaud 2017-03-16 12:52:53 CET
Don't bother, I'll just fix my local commit.
Comment 27 Marja Van Waes 2017-03-16 16:30:02 CET
(In reply to Thierry Vignaud from comment #26)
> Don't bother, I'll just fix my local commit.

Thanks, Thierry :-)

Here's another one https://ml.mageia.org/l/arc/i18n-discuss/2017-03/msg00026.html

Yochenhsieh mentions:

> for zh_tw:
>
> msgid "Normal mode"
>
> msgstr "ä¸è¬æ¨¡å¼"
Comment 28 Thierry Vignaud 2017-03-16 17:46:58 CET
OK, I removed 2 extra characters
Let's blame google translator's validation :-)

Keywords: (none) => PATCH

Comment 29 Mageia Robot 2017-03-17 02:14:39 CET
commit 8e5160b818c6d4f25ca7aa5d72db6d5754b92334
Author: Thierry Vignaud <thierry.vignaud@...>
Date:   Wed Mar 15 07:48:29 2017 +0100

    shrink "expert/normal mode" string
    
    Thus fixing too big translations that make the partition window to be
    too big (mga#20360)
    This was a side effect of commit 947ab9bb0fb06611b5bd54465f98545ff175afb2
    
    Adjust translations accordingly
    (note that some translators have already done so)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=8e5160b818c6d4f25ca7aa5d72db6d5754b92334
Comment 30 Mageia Robot 2017-03-17 02:14:42 CET
commit 784a9155115058cbb494506681178d4af5f0d956
Author: Thierry Vignaud <thierry.vignaud@...>
Date:   Wed Mar 15 07:51:11 2017 +0100

    do not make all action buttons the same size
    
    Thus fixing too big translations that make the partition window to be
    too big (mga#20360)
    This was a side effect of commit 947ab9bb0fb06611b5bd54465f98545ff175afb2
    which itself partially reverted commit 8e187d972ed59909271d48915eef4af34e7fe068
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=784a9155115058cbb494506681178d4af5f0d956
Comment 31 Thierry Vignaud 2017-03-17 02:18:03 CET
Closing

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 32 Mageia Robot 2017-03-17 08:07:15 CET
commit 464abafd8ae6e441a9c82b10d32c0543c9149eb6
Author: Marja van Waes <marja@...>
Date:   Fri Mar 17 08:06:41 2017 +0100

    change 'Normal mode' string exactly to what yochenhsieh proposed (mga#20360)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=464abafd8ae6e441a9c82b10d32c0543c9149eb6
Comment 33 Thierry Vignaud 2017-03-27 14:32:30 CEST
*** Bug 20575 has been marked as a duplicate of this bug. ***

CC: (none) => vigenmailbox

Comment 34 Thierry Vignaud 2017-04-04 11:55:09 CEST
(In reply to psyca from comment #23)
> I have got something similar also in the normal partitioning screen (see
> english screenshot).
> I think its because Windows marked the first partition as
> "System-reserviert" (system reserved).
> As you can see in the screenshot, the filesystem description "Empty" leaves
> the screen (Emp). Maybe the whole Harddrive partition bar is also to long
> (?).

For this, we could use ellipses for the wizard too
See http://gitweb.mageia.org/software/drakx/commit/?id=5299ba86
But please open another bug report for that
Thierry Vignaud 2017-04-04 12:02:45 CEST

Blocks: (none) => 20624


Note You need to log in before you can comment on or make changes to this bug.