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

[wine-bugs at winehq.org]

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

            Bug ID: 35312
           Summary: "Toggle All" currently means "Inverse Selection"
           Product: Wine-Testbot
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: unknown
          Assignee: wine-bugs at winehq.org
          Reporter: 00cpxxx at gmail.com
    Classification: Unclassified

When selecting VMs the button "Toggle All" actually inverses the current
selection.

function ToggleAll()
{
  for (var i = 0; i < document.forms[0].elements.length; i++)
  {
    if(document.forms[0].elements[i].type == 'checkbox')
      document.forms[0].elements[i].checked =
!(document.forms[0].elements[i].checked);
  }
}

A possible solution:

var toggledstate = true;
function ToggleAll()
{
  toggledstate = !toggledstate;
  for (var i = 0; i < document.forms[0].elements.length; i++)
  {
    if(document.forms[0].elements[i].type == 'checkbox')
      document.forms[0].elements[i].checked = toggledstate;
  }
}

-- 
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.