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

DropDown is loosing focus on selection of an option

    Details

    • Similar Issues:
    • Epic Link:
    • Sprint:
      Blue Ocean 1.4 - beta 1, Blue Ocean 1.4 - beta 3, Blue Ocean 1.4 - beta 2, Blue Ocean 1.5 - beta 1

      Description

      Update: I just tested with firefox and chrome and the loose the focus in showing there as well.

      When using IE and selecting DropDown options with select it is not possible to focus the next element in the tabIndex since the focus got lost.

      Steps to reproduce:
      Either http://jenkinsci.github.io/jenkins-design-language/docs (someone may want to update the docs and maybe correct the url where deployed (linked to ) or in your local jdl storybook.

      You may need to "patch" IE and add in css height=800px to dislay div otherwise you see a too small box.

      • search for Dropdown
      • press ENTER on the selection button
      • when the selection list comes, press ENTER on any given option
      • the result is that you cannot focus the next element anymore. IE has "ADD" and lost focus

        When you use "document.activeElement" on the storybook page the iframe should be the result and in a jenkins instance NULL is the result.

        Further ESC and TAB as well are loosing focus.

      In our project I had a similar problem where IE gave focus on a element that did not exist anymore and therefore uses NULL

        Attachments

          Issue Links

            Activity

            Hide
            kshultz Karl Shultz added a comment -

            Testing Notes:

            Because this is a really specific set of conditions (IE11 user, plus Blue Ocean, plus keyboard navigation), manual verification of this one, once fixed, seems appropriate.

            Show
            kshultz Karl Shultz added a comment - Testing Notes : Because this is a really specific set of conditions (IE11 user, plus Blue Ocean, plus keyboard navigation), manual verification of this one, once fixed, seems appropriate.
            Hide
            tscherler Thorsten Scherler added a comment -

            I just tested with firefox and chrome and the loose the focus in showing there as well.

            Show
            tscherler Thorsten Scherler added a comment - I just tested with firefox and chrome and the loose the focus in showing there as well.
            Hide
            tscherler Thorsten Scherler added a comment -

            Karl Shultz regarding testing notes I just found out that it applies to any browser. I further think an unit test should be covering the code change.

            Show
            tscherler Thorsten Scherler added a comment - Karl Shultz regarding testing notes I just found out that it applies to any browser. I further think an unit test should be covering the code change.
            Hide
            tscherler Thorsten Scherler added a comment -

            shows:

            • select dropdown with mose
            • select an option
            • hit enter again
            • nothing happens because the button lost focus
            • you have to press TAB to get focus again
            Show
            tscherler Thorsten Scherler added a comment - shows: select dropdown with mose select an option hit enter again nothing happens because the button lost focus you have to press TAB to get focus again
            Hide
            alobato Alvaro Lobato added a comment -

            Vivek Pandey can we backport this to 1.4.x ?

            Show
            alobato Alvaro Lobato added a comment - Vivek Pandey can we backport this to 1.4.x ?
            Hide
            vivek Vivek Pandey added a comment -

            Alvaro Lobato Sorry, somehow I missed it. PR is opened, https://github.com/jenkinsci/blueocean-plugin/pull/1660 for back port. I should be able to cut 1.4.2 release today after its merged.

            Show
            vivek Vivek Pandey added a comment - Alvaro Lobato Sorry, somehow I missed it. PR is opened, https://github.com/jenkinsci/blueocean-plugin/pull/1660  for back port. I should be able to cut 1.4.2 release today after its merged.
            Hide
            vivek Vivek Pandey added a comment -

            Alvaro Lobato Ported and 1.4.2 released.

            Show
            vivek Vivek Pandey added a comment - Alvaro Lobato Ported and 1.4.2 released.

              People

              • Assignee:
                nicu Nicolae Pascu
                Reporter:
                tscherler Thorsten Scherler
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: