[Bug 35312] "Toggle All" currently means "Inverse Selection"

[wine-bugs at winehq.org]

https://bugs.winehq.org/show_bug.cgi?id=35312

François Gouget <fgouget at codeweavers.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fgouget at codeweavers.com

--- Comment #1 from François Gouget <fgouget at codeweavers.com> ---
I'd argue that 'Toggle' means 'inverse the current state'. So toggling a switch
does not mean turning it on but setting it to the other position, which may
either be on or off.

But then I may agree with you that 'toggle' is not necessarily what we want
here. It may make more sense to have something that checks everything if one
box was unchecked, and otherwise unchecks everything. I believe that's how
gmail's global selection checkbox works for instance.

Finally, regarding the proposed code I don't think using a global variable is a
good idea. Fortunately it should not be needed.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.