[f20/master] Use ceil for minSize in resize dialog (#1040012)
Vratislav Podzimek
vpodzime at redhat.com
Wed Dec 18 12:30:15 UTC 2013
On Wed, 2013-12-11 at 11:54 -0800, Brian C. Lane wrote:
> From: "Brian C. Lane" <bcl at redhat.com>
>
> ---
> pyanaconda/ui/gui/spokes/lib/resize.py | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/pyanaconda/ui/gui/spokes/lib/resize.py b/pyanaconda/ui/gui/spokes/lib/resize.py
> index ee5cdbe..d9ef00b 100644
> --- a/pyanaconda/ui/gui/spokes/lib/resize.py
> +++ b/pyanaconda/ui/gui/spokes/lib/resize.py
> @@ -21,6 +21,7 @@
>
> from __future__ import division
> from collections import namedtuple
> +from math import ceil
>
> from gi.repository import Gdk, Gtk
>
> @@ -227,27 +228,27 @@ class ResizeDialog(GUIObject):
> with here.
> """
> self._resizeSlider.handler_block_by_func(self.on_resize_value_changed)
> - self._resizeSlider.set_range(device.minSize, int(device.size))
> + self._resizeSlider.set_range(int(ceil(device.minSize)), int(device.size))
> self._resizeSlider.handler_unblock_by_func(self.on_resize_value_changed)
> self._resizeSlider.set_value(value)
>
> # The slider needs to be keyboard-accessible. We'll make small movements change in
> # 1% increments, and large movements in 5% increments.
> - distance = int(device.size) - int(device.minSize)
> + distance = int(device.size) - int(ceil(device.minSize))
> onePercent = distance*0.01
> fivePercent = distance*0.05
> twentyPercent = distance*0.2
>
> adjustment = self.builder.get_object("resizeAdjustment")
> - adjustment.configure(value, device.minSize, int(device.size), onePercent, fivePercent, 0)
> + adjustment.configure(value, int(ceil(device.minSize)), int(device.size), onePercent, fivePercent, 0)
>
> # And then the slider needs a couple tick marks for easier navigation.
> self._resizeSlider.clear_marks()
> for i in range(1, 5):
> - self._resizeSlider.add_mark(device.minSize + i*twentyPercent, Gtk.PositionType.BOTTOM, None)
> + self._resizeSlider.add_mark(int(ceil(device.minSize)) + i*twentyPercent, Gtk.PositionType.BOTTOM, None)
>
> # Finally, add tick marks for the ends.
> - self._resizeSlider.add_mark(device.minSize, Gtk.PositionType.BOTTOM, size_str(device.minSize))
> + self._resizeSlider.add_mark(int(ceil(device.minSize)), Gtk.PositionType.BOTTOM, size_str(int(ceil(device.minSize))))
> self._resizeSlider.add_mark(int(device.size), Gtk.PositionType.BOTTOM, size_str(int(device.size)))
>
> def _update_action_buttons(self, row):
Cherry-picked to master together with my f20-branch-only patches so that
we don't have a regression in F21. Let's hope they will all be replaced
with a better solution by that time.
--
Vratislav Podzimek
Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic
More information about the anaconda-patches
mailing list