Bug 15728 - it is invisible in the doPartitionDisk screen, that the windows partition can be resized there.
Summary: it is invisible in the doPartitionDisk screen, that the windows partition can...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: Installer (show other bugs)
Version: Cauldron
Hardware: x86_64 Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Pascal Terjan
QA Contact:
URL:
Whiteboard: 5RC
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-17 21:09 CEST by Marja Van Waes
Modified: 2015-04-22 09:42 CEST (History)
5 users (show)

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


Attachments
doPartitionDisk screen (208.61 KB, image/png)
2015-04-17 21:09 CEST, Marja Van Waes
Details
Untested patch (2.06 KB, patch)
2015-04-20 01:39 CEST, Pascal Terjan
Details | Diff
Small improvement (108.28 KB, image/png)
2015-04-20 22:49 CEST, Pascal Terjan
Details
New version (108.43 KB, image/png)
2015-04-22 00:51 CEST, Pascal Terjan
Details

Description Marja Van Waes 2015-04-17 21:09:51 CEST
Created attachment 6307 [details]
doPartitionDisk screen

When pterjan created that screen, long long ago, it was visible that the windows partition could be resized
https://farm3.staticflickr.com/2429/4025954643_3de81b40fb_o.png

However, after already becoming as good as invisible 
http://docteam.mageia.nl/en/installer/content/images/dx2-doPartitionDisks.png
it has now become totally invisible: windows and mageia are glued together, without division between them.
Comment 1 Marja Van Waes 2015-04-17 21:10:32 CEST
the attachment is how it currently looks, with Adwaita
Marja Van Waes 2015-04-17 22:45:32 CEST

Whiteboard: (none) => 5RC

Pascal Terjan 2015-04-17 23:19:39 CEST

Status: NEW => ASSIGNED
Assignee: bugsquad => pterjan

Comment 2 Pascal Terjan 2015-04-20 01:39:14 CEST
Created attachment 6318 [details]
Untested patch
Comment 3 Marja Van Waes 2015-04-20 08:54:41 CEST
(In reply to Pascal Terjan from comment #2)
> Created attachment 6318 [details]
> Untested patch

I tried to patch drakx-installer-stage2-16.84-1 in SVN with it, but it fails.

+ /usr/bin/patch -U -s -p1 -b --suffix .0001 --fuzz=0 -i /home/marja/home2/marja/svn/drakx-installer-stage2/SOURCES/0001-show-resize-handle.patch
1 out of 3 hunks FAILED -- saving rejects to file perl-install/fs/partitioning_wizard.pm.rej

perl-install/fs/partitioning_wizard.pm.rej contains:


--- perl-install/fs/partitioning_wizard.pm
+++ perl-install/fs/partitioning_wizard.pm
@@ -305,14 +305,12 @@ sub create_display_box {

     my $display_box = ugtk3::gtkset_size_request(Gtk3::HBox->new(0,0), -1, 26);

-    my $sep_count = @parts - 1;
     #- ratio used to compute initial partition pixel width (each partition should be > min_width)
     #- though, the pixel/sectors ratio cannot be the same for all the partitions
-    my $initial_ratio = $totalsectors ? ($width - @parts * $minwidth - $sep_count) / $totalsectors : 1;
+    my $initial_ratio = $totalsectors ? ($width - @parts * $minwidth) / $totalsectors : 1;

     my $vbox = Gtk3::VBox->new;

-    my $part_sep;
     my $desc;

     if ($resize) {
~
Comment 4 Marja Van Waes 2015-04-20 09:15:07 CEST
ah, the patch doesn't work because it is made against the git version of perl-install/fs/partitioning_wizard.pm ?

Will try again tonight
Comment 5 Pascal Terjan 2015-04-20 16:27:58 CEST
I'll test it properly tonight, I'm not even sure this method exists :)
Comment 6 Marja Van Waes 2015-04-20 21:34:48 CEST
(In reply to Pascal Terjan from comment #5)
> I'll test it properly tonight, I'm not even sure this method exists :)

lol

Thanks, I'm happy to leave the testing to you :-)
Comment 7 Pascal Terjan 2015-04-20 22:49:16 CEST
Created attachment 6320 [details]
Small improvement

I fixed the separator between other partitions and made this one larger but this is still not enough to make it visible.

I tried adding an image with the 3 dots but it caused the stage2 to crash immediately :(
Comment 8 Marja Van Waes 2015-04-21 07:29:54 CEST
(In reply to Pascal Terjan from comment #7)
> Created attachment 6320 [details]
> Small improvement
> 
> I fixed the separator between other partitions and made this one larger but
> this is still not enough to make it visible.
> 
> I tried adding an image with the 3 dots but it caused the stage2 to crash
> immediately :(

Thanks for having tried, and for already improving it this far :-)

Does
"- GtkWindow no longer supports resize grips" in this message about gtk+ 3.13.4 release:
https://mail.gnome.org/archives/ftp-release-list/2014-July/msg00029.html
mean there is no longer a normal way to do this?

CC'ing ovitters, in the hope he can maybe help.

CC: (none) => olav

Comment 9 Marja Van Waes 2015-04-21 08:02:21 CEST
Is it possible to turn the grey of the separator into black and white dots? (Can e.g. the resolution of only that seperator be set to a very low value?)

If so, would that look more like a grip?
Comment 10 Pascal Terjan 2015-04-22 00:51:52 CEST
Created attachment 6322 [details]
New version
Comment 11 Mageia Robot 2015-04-22 00:54:59 CEST
commit 4e8627d6cba69fbfa8ed574f83d3722b0bdf6e49
Author: Pascal Terjan <pterjan@...>
Date:   Tue Apr 21 21:27:36 2015 +0000

    Make the resize handler more visible (mga#15728)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=4e8627d6cba69fbfa8ed574f83d3722b0bdf6e49
Comment 12 Pascal Terjan 2015-04-22 01:06:49 CEST
Closing

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

Comment 13 Marja Van Waes 2015-04-22 07:57:40 CEST
That looks really good! Thanks Pascal
Comment 14 Rémi Verschelde 2015-04-22 08:35:06 CEST
Nice job, it's indeed much clearer this way.

CC: (none) => rverschelde

Comment 15 Mageia Robot 2015-04-22 09:42:35 CEST
commit 8a3e0a2b8b39048f5c4727cf408239a257737907
Author: Pascal Terjan <pterjan@...>
Date:   Tue Apr 21 21:27:36 2015 +0000

    Make the resize handler more visible (mga#15728)
---
 Commit Link:
   http://gitweb.mageia.org/software/drakx/commit/?id=8a3e0a2b8b39048f5c4727cf408239a257737907

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