Disabling upload/download limits shows '-1'

[Matt Griffin]

Showing '-1' is confusing to users. Instead don't show any value.

[Natalia Bidart (nataliabidart)]

Hi Matt!

Thanks for the feedback. Since we're using a spinbox, we can not "not show any value". We could improve this somehow (can't think of a good and fast way ATM). I will assign the design team to this bug so we can have some help from them.

Thanks!

[Leo Arias (elopio)]

Why don't you just disable the spinbox?
It would also be boring to uncheck the limits temporarily and then get back from -1 to the value we frequently use.

[Natalia Bidart (nataliabidart)]

Well, is very very tricky in the backend to do this, since syncdaemon will not allow you to enable/disable a single limit (either up or down), but only the 2 simultaneously (either both are off or both are on).

So, as a workaround for the original design, when the user decides to throttle a single limit, we enable throttling for all limits but set the not-throttled limit to a very high value.

[Matt Griffin (mattgriffin)]

Naty: Maybe the solution is to offer a single controller for both up and downstream rates and put it in natural language? The downside is that the text will need to be translated. But this is covered in bug #847233.

[Natalia Bidart (nataliabidart)]

Matt,

I would love that solution (in fact is what we have in the GTK control panel), but when we first discussed it with the design team they requested this approach (back in May, at Budapest).

So, at this point, we're in a sort of UI freeze, and unless we have a serious usability bug, I'm not sure is a good idea to change UI elements and designs.

Thanks a lot for the effort you both are putting in this!

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntuone-control-panel - 3.99.0-0ubuntu1

---------------
ubuntuone-control-panel (3.99.0-0ubuntu1) quantal; urgency=low

  * New upstream release.
    - Improve upload/download limit UI. (LP: #847227, #985484)
    - Fix sorting of shares and folders. (LP: #1006385)
    - Use suggested path for server path comparisons. (LP: #1005574)
  * debian/control:
    - Update build dependencies in preparation for running tests during builds.
  * debian/watch:
    - Update to use stable-4-0 series for Quantal.
 -- Rodney Dawes <email address hidden> Wed, 20 Jun 2012 16:42:13 -0400