• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • None
    • Jenkins 2.260
      Pipeline stage view 2.16
      Firefox 81.0.1

      After upgrading pipeline stage view plugin to version 2.16 the input stage "hover box" no longer appear for input stages. Input stages could still be completed using the links in "Console Output".

      This screenshot was taken after downgrading to 2.15. It's the yellow box I mean was missing in 2.16.

        1. jenkins-scrollbar.png
          jenkins-scrollbar.png
          172 kB
        2. screenshot-1.png
          screenshot-1.png
          10 kB
        3. JenkinsInputLocation.png
          JenkinsInputLocation.png
          59 kB
        4. jenkins_input_hover.png
          jenkins_input_hover.png
          41 kB

          [JENKINS-63892] Missing pipeline input hover in stage view

          Robin Karlsson created issue -
          Ed Thorne made changes -
          Attachment New: JenkinsInputLocation.png [ 52812 ]

          Ed Thorne added a comment - - edited

          It's actually there, it's just not where it's supposed to be. In my case the first approval wasn't visible. I suspect it rendered but out of frame. The second approval showed up on the far left side of the browser as pictured below.

          I suspect this may be related to the recent change in pull request 107.

          Ed Thorne added a comment - - edited It's actually there, it's just not where it's supposed to be. In my case the first approval wasn't visible. I suspect it rendered but out of frame. The second approval showed up on the far left side of the browser as pictured below. I suspect this may be related to the recent change in pull request 107 .

          Nice find! So a workaround could be building longer pipelines!

          Robin Karlsson added a comment - Nice find! So a workaround could be building longer pipelines!

          Ed Thorne added a comment - - edited

          Actually the PR author has already pushed a fix in pull request 116 so the next version of the plugin should fix the issue. I tested it by altering the in memory CSS based on his PR and the popover appears where you'd expect it.

          Ed Thorne added a comment - - edited Actually the PR author has already pushed a fix in pull request 116 so the next version of the plugin should fix the issue. I tested it by altering the in memory CSS based on his PR and the popover appears where you'd expect it.

          Jimmy Praet added a comment -

          The input hover is still not showing up correctly for me in 2.17.

          Jimmy Praet added a comment - The input hover is still not showing up correctly for me in 2.17.

          Same here. Even with 2.17 the box is not showing up.

          And if I check the Style Editor in Firefox it's as if pull request 116 wasn't included in 2.17. I see:

          .cbwf-widget {
            width: calc(100% - 10px);
          }
          

          and if I change it to

          .cbwf-widget {
            width: calc(~'100% - 10px');
          }
          

          the box appears where it should!

          I've tried reloading hard and clearing cache, but no luck.

          Robin Karlsson added a comment - Same here. Even with 2.17 the box is not showing up. And if I check the Style Editor in Firefox it's as if pull request 116 wasn't included in 2.17. I see: .cbwf-widget { width : calc(100% - 10px); } and if I change it to .cbwf-widget { width : calc(~ '100% - 10px' ); } the box appears where it should! I've tried reloading hard and clearing cache, but no luck.
          Tobias Gruetzmacher made changes -
          Link New: This issue is duplicated by JENKINS-63901 [ JENKINS-63901 ]
          Tobias Gruetzmacher made changes -
          Link New: This issue is duplicated by JENKINS-63914 [ JENKINS-63914 ]

          snago Since ~'...' is invalid in plain CSS (that escaping was needed for LESS), changing that effectively disables the width property, which makes the popup return to its rightful place...

          The problem seems to be that the old JS code and the new CSS styles are not equivalent: The old code was only applied to the whole stage view (as far as I can see) and the new CSS is also applied to the input popup. (AFAICS, I'm not an expert on CSS)

          Tobias Gruetzmacher added a comment - snago Since ~'...' is invalid in plain CSS (that escaping was needed for LESS), changing that effectively disables the width property, which makes the popup return to its rightful place... The problem seems to be that the old JS code and the new CSS styles are not equivalent: The old code was only applied to the whole stage view (as far as I can see) and the new CSS is also applied to the input popup. (AFAICS, I'm not an expert on CSS)

            Unassigned Unassigned
            snago Robin Karlsson
            Votes:
            14 Vote for this issue
            Watchers:
            16 Start watching this issue

              Created:
              Updated:
              Resolved: