Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-69110

Plugin manager "select all" checkbox doesn't change state on click

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • 2.360
    • 2.361

      Something changed between the time https://github.com/jenkinsci/jenkins/pull/6806 was reviewed and merged and 2.360 was released, because the current implemenation doesn't match what the PR shows and what we reviewed.

      See the attachment for the current design.

      Browsertools complains about the following when clicking the checkbox to select all plugins:

      index.js:49 Uncaught TypeError: Cannot read properties of null (reading 'classList')
          at updateIcon (index.js:49:25)
          at HTMLButtonElement.<anonymous> (index.js:33:5)
      updateIcon @ index.js:49
      (anonymous) @ index.js:33
      

      Do you mind taking a look janfaracik?

          [JENKINS-69110] Plugin manager "select all" checkbox doesn't change state on click

          Alexander Brandes created issue -
          Jan Faracik made changes -
          Assignee New: Jan Faracik [ janfaracik ]
          Jan Faracik made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Jan Faracik made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]

          Jan Faracik added a comment -

          Opened a fix for that here https://github.com/jenkinsci/jenkins/pull/6911 - thanks for raising  

          Jan Faracik added a comment - Opened a fix for that here https://github.com/jenkinsci/jenkins/pull/6911  - thanks for raising  
          Alexander Brandes made changes -
          Remote Link New: This issue links to "https://github.com/jenkinsci/jenkins/pull/6911 (Web Link)" [ 27993 ]
          Basil Crow made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Fixed but Unreleased [ 10203 ]
          Alexander Brandes made changes -
          Released As New: 2.361
          Status Original: Fixed but Unreleased [ 10203 ] New: Closed [ 6 ]

            janfaracik Jan Faracik
            notmyfault Alexander Brandes
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: