In the project view (job page) the links below the last trend graph are not clickable since the footer overlaps these links. It happens only if you have several trend graphs so you need to scroll the page in order to view the bottom one.

      In order to reproduce this bug, the test case could be adapted so that the analysis-collector plug-in is running this test.

          [JENKINS-30304] Footer hides trend graph links

          Ulli Hafner created issue -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/org/jenkinsci/test/acceptance/po/Job.java
          src/test/java/plugins/AbstractAnalysisTest.java
          http://jenkins-ci.org/commit/acceptance-test-harness/e38955e49e1ef6d147d8b09191d59674e46022f3
          Log:
          JENKINS-29900 Improved test so that detail trend graph is verified, too.

          Still has a workaround for JENKINS-30304.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/org/jenkinsci/test/acceptance/po/Job.java src/test/java/plugins/AbstractAnalysisTest.java http://jenkins-ci.org/commit/acceptance-test-harness/e38955e49e1ef6d147d8b09191d59674e46022f3 Log: JENKINS-29900 Improved test so that detail trend graph is verified, too. Still has a workaround for JENKINS-30304 .
          Ulli Hafner made changes -
          Description Original: In the project view (job page) the links below the last trend graph are not clickable since the footer overlaps these links. It happens only if you have several trend graphs so you need to scroll the page in order to view the bottom one. New: In the project view (job page) the links below the last trend graph are not clickable since the footer overlaps these links. It happens only if you have several trend graphs so you need to scroll the page in order to view the bottom one.

          In order to reproduce this bug, the test case https://github.com/jenkinsci/acceptance-test-harness/blob/e38955e49e1ef6d147d8b09191d59674e46022f3/src/test/java/plugins/AbstractAnalysisTest.java#L120 could be adapted so that the analysis-collector plug-in is running this test.
          Ulli Hafner made changes -
          Description Original: In the project view (job page) the links below the last trend graph are not clickable since the footer overlaps these links. It happens only if you have several trend graphs so you need to scroll the page in order to view the bottom one.

          In order to reproduce this bug, the test case https://github.com/jenkinsci/acceptance-test-harness/blob/e38955e49e1ef6d147d8b09191d59674e46022f3/src/test/java/plugins/AbstractAnalysisTest.java#L120 could be adapted so that the analysis-collector plug-in is running this test.
          New: In the project view (job page) the links below the last trend graph are not clickable since the footer overlaps these links. It happens only if you have several trend graphs so you need to scroll the page in order to view the bottom one.

          In order to reproduce this bug, the [test case|https://github.com/jenkinsci/acceptance-test-harness/blob/e38955e49e1ef6d147d8b09191d59674e46022f3/src/test/java/plugins/AbstractAnalysisTest.java#L120] could be adapted so that the analysis-collector plug-in is running this test.
          Manuel Recena Soto made changes -
          Assignee New: Manuel Recena Soto [ recena ]

          drulli, Could you provide the OS and web browser (and version)?

          Manuel Recena Soto added a comment - drulli , Could you provide the OS and web browser (and version)?

          Ulli Hafner added a comment -

          I noticed that on OSX with either latest firefox and chrome while writing an acceptance test for the analysis-plug-in. You can remove the if in https://github.com/jenkinsci/acceptance-test-harness/blob/master/src/test/java/plugins/AbstractAnalysisTest.java#L173 and then the test will fail for the analysis-collector plug-in. Or should I prepare the test case for you?

          Ulli Hafner added a comment - I noticed that on OSX with either latest firefox and chrome while writing an acceptance test for the analysis-plug-in. You can remove the if in https://github.com/jenkinsci/acceptance-test-harness/blob/master/src/test/java/plugins/AbstractAnalysisTest.java#L173 and then the test will fail for the analysis-collector plug-in. Or should I prepare the test case for you?

          drulli, I think this PR solves the issue.

          Manuel Recena Soto added a comment - drulli , I think this PR solves the issue.
          Manuel Recena Soto made changes -
          Remote Link New: This issue links to "PR (Web Link)" [ 13527 ]

          drulli, Should this method be like this?

          private void verifyTrendGraphDetails(final FreeStyleJob job, final AnalysisAction action) {
              List<WebElement> graphLinks = job.all(By.linkText("Enlarge"));
              for (int i = 0; i < graphLinks.size(); i++) {
                  graphLinks.get(i).click();
                  assertThatProjectPageTrendIsCorrect(job, action, "../../");
              }
          }
          

          Manuel Recena Soto added a comment - drulli , Should this method be like this? private void verifyTrendGraphDetails( final FreeStyleJob job, final AnalysisAction action) { List<WebElement> graphLinks = job.all(By.linkText( "Enlarge" )); for ( int i = 0; i < graphLinks.size(); i++) { graphLinks.get(i).click(); assertThatProjectPageTrendIsCorrect(job, action, "../../" ); } }

            recena Manuel Recena Soto
            drulli Ulli Hafner
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: