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

Horizontal scrolling & overflow issues

    Details

    • Similar Issues:

      Description

      Tests using matrix authorization (or permissions tables similar to the used in this configuration) are presenting some issues relates to scrolling.

      I have only been able to reproduce it in Mac (in a Linux VM seems to work) but when clicking in permissions outside the current view, the scrolling the element is not checked (but no error is launched), causing the test to fail downstream when the expected permission is not set.

      Besides, tests in which elements with specific overflow styles (such as the "Add Credentials Dialog", used for example in CreateSlaveTest#newSlave are also problematic.

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Andrew Bayer
            Path:
            src/main/java/org/jenkinsci/test/acceptance/selenium/Scroller.java
            src/main/resources/org/jenkinsci/test/acceptance/selenium/scroller.js
            src/test/java/plugins/MatrixAuthPluginTest.java
            http://jenkins-ci.org/commit/acceptance-test-harness/1cdd0988e3175f929f784366cdb71398d916341b
            Log:
            Revert "JENKINS-34411 Account also for horizontal scrolling"

            This reverts commit 254d0151d26d3bf5e71e891650c2303b49b91d49.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Bayer Path: src/main/java/org/jenkinsci/test/acceptance/selenium/Scroller.java src/main/resources/org/jenkinsci/test/acceptance/selenium/scroller.js src/test/java/plugins/MatrixAuthPluginTest.java http://jenkins-ci.org/commit/acceptance-test-harness/1cdd0988e3175f929f784366cdb71398d916341b Log: Revert " JENKINS-34411 Account also for horizontal scrolling" This reverts commit 254d0151d26d3bf5e71e891650c2303b49b91d49.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Andrew Bayer
            Path:
            src/main/java/org/jenkinsci/test/acceptance/selenium/Scroller.java
            src/main/resources/org/jenkinsci/test/acceptance/selenium/overflow.js
            src/main/resources/org/jenkinsci/test/acceptance/selenium/scroller.js
            src/test/java/plugins/MatrixAuthPluginTest.java
            http://jenkins-ci.org/commit/acceptance-test-harness/e5ba520cd204f719036ebd1665cadc3b91228366
            Log:
            Merge pull request #104 from abayer/revert-jenkins-34411

            Revert JENKINS-34411

            Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/e11f1649c699...e5ba520cd204

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Bayer Path: src/main/java/org/jenkinsci/test/acceptance/selenium/Scroller.java src/main/resources/org/jenkinsci/test/acceptance/selenium/overflow.js src/main/resources/org/jenkinsci/test/acceptance/selenium/scroller.js src/test/java/plugins/MatrixAuthPluginTest.java http://jenkins-ci.org/commit/acceptance-test-harness/e5ba520cd204f719036ebd1665cadc3b91228366 Log: Merge pull request #104 from abayer/revert-jenkins-34411 Revert JENKINS-34411 Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/e11f1649c699...e5ba520cd204
            Hide
            abayer Andrew Bayer added a comment -

            fwiw, CreateSlaveTest.newSlave seems to be ok now with Credentials plugin 2.0.6...

            Show
            abayer Andrew Bayer added a comment - fwiw, CreateSlaveTest.newSlave seems to be ok now with Credentials plugin 2.0.6...
            Hide
            abayer Andrew Bayer added a comment -

            Nope, that's just my local env due to the larger window size!

            Show
            abayer Andrew Bayer added a comment - Nope, that's just my local env due to the larger window size!
            Hide
            olivergondza Oliver Gondža added a comment -

            Is there something else to do?

            Show
            olivergondza Oliver Gondža added a comment - Is there something else to do?

              People

              • Assignee:
                andresrc Andres Rodriguez
                Reporter:
                andresrc Andres Rodriguez
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: