Re: [Tails-dev] Please review and merge feature/liveusb_ui_…

Nachricht löschen

Nachricht beantworten
Autor: Alan
Datum:  
To: tails-dev
Neue Treads: Re: [Tails-dev] feature/liveusb_ui_improvement followups
Betreff: Re: [Tails-dev] Please review and merge feature/liveusb_ui_improvement
Hi,

On Sat, 12 Oct 2013 09:36:52 +0200 intrigeri <intrigeri@???> wrote:
> Alan wrote (11 Oct 2013 17:52:19 GMT) :
> >> Just curious: any specific reason that I've missed to display the
> >> device size in the confirmation dialog on initial install, but not
> >> at upgrade time?
> >>
> > No.
>
> Do you intend to fix this (say, in time for 0.22), or should I?
>

I'll do it in time for 0.22.

> >> > +def _format_bytes(value):
> >> > +    return '%0.1f GB' % (value / 10.0**9)

> >>
> >> I appreciate the minor refactoring, but then I'd rather:
> >>
> >>   * either use some library to display human-readable units in all
> >>     cases (think: 2TB external drive) to improve UX
> >>   * or simply rename the function to _format_bytes_in_GB, so that
> >> at least the name expresses its current limitations.

> >>
> > I would choose the 2nd for now.
>
> Do you intend to fix this (say, in time for 0.22), or should I?


I'll.
>
> >> However, the No/Yes buttons ordering seems unusual to me. I didn't
> >> dare patching it, as I thought you might have followed the GNOME
> >> HIG or something, and I may be missing something. Any reason to
> >> order it this way?
> >>
> > No. I'm used to GTK+, which chooses the right order automatically.
> > Qt might not do that.
>
> Do you intend to fix this (say, in time for 0.22), or should I?
>

I'll too.

> >> To end with, the test suite was not updated yet, and is likely
> >> broken by these changes, so please file a ticket about it,
> >> assigned to me if you wish (I'd rather do it together, though).
> >> This must be fixed before the 0.21 freeze.
> >>
> > I'm afraid that I don't have the infrastructure to do that yet, so I
> > would appreciate if you took care of it.
>
> OK, will (try to) do.
>

Thanks.

Cheers