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

Developer sees ETAs on hover states for all indicator types

    XMLWordPrintable

    Details

    • Similar Issues:
    • Sprint:
      iapetus

      Description

      In Scope

      • Hovering over each node shows the following tooltip
        • Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
        • Successful – text: "Passed in 2 minutes and 23 seconds"
        • Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
        • Failed – text: "Failed in 2 minutes and 23 seconds"
        • Not built – text: "Estimated to start in 2 minutes 32 seconds"
        • Unknown – text: "Unknown"
      • All text must be i18n'able

        Attachments

          Issue Links

            Activity

            jamesdumay James Dumay created issue -
            jamesdumay James Dumay made changes -
            Field Original Value New Value
            Link This issue is blocked by UX-391 [ UX-391 ]
            jamesdumay James Dumay made changes -
            Epic Link UX-79 [ 25277 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Description In Scope
            * Successful – text: "Passed"
            * Unstable – text: "Passed but has been determined unstable"
            * Failed – text: "Failed"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            In Scope
            * Use the tooltip defined in UX-391
            * Successful – text: "Passed"
            * Unstable – text: "Passed but has been determined unstable"
            * Failed – text: "Failed"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            jamesdumay James Dumay made changes -
            Description In Scope
            * Use the tooltip defined in UX-391
            * Successful – text: "Passed"
            * Unstable – text: "Passed but has been determined unstable"
            * Failed – text: "Failed"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            In Scope
            * Use the tooltip defined in UX-391
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed 2 minutes and 23 seconds and marked as unstable."
            * Failed – text: "Failed"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            jamesdumay James Dumay made changes -
            Description In Scope
            * Use the tooltip defined in UX-391
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed 2 minutes and 23 seconds and marked as unstable."
            * Failed – text: "Failed"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            In Scope
            * Use the tooltip defined in UX-391
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed 2 minutes and 23 seconds and marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            jamesdumay James Dumay made changes -
            Description In Scope
            * Use the tooltip defined in UX-391
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed 2 minutes and 23 seconds and marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            In Scope
            * Use the tooltip defined in UX-391
            * Should be done for regular indicators and the indicators in the railroad
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed 2 minutes and 23 seconds and marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            Hide
            jamesdumay James Dumay added a comment -

            jmcdonald here's where the indicator implementations being different becomes a problem... I feel like we will be chasing our tails to make sure they behave the same way.

            Show
            jamesdumay James Dumay added a comment - jmcdonald here's where the indicator implementations being different becomes a problem... I feel like we will be chasing our tails to make sure they behave the same way.
            jamesdumay James Dumay made changes -
            Link This issue is blocked by UX-385 [ UX-385 ]
            jamesdumay James Dumay made changes -
            Link This issue is duplicated by UX-375 [ UX-375 ]
            jamesdumay James Dumay made changes -
            Sprint 1.0-m7 [ 61 ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Link This issue is blocked by UX-399 [ UX-399 ]
            Hide
            sophistifunk Josh McDonald added a comment -

            Yes, we expected it to be a problem at some point. But if we're going to build in complicated hover popups, then uniting them into a shared componet that can function as both (part of a larger SVG) and (a first-class HTML DOM node) will become more complicated, as well.

            Show
            sophistifunk Josh McDonald added a comment - Yes, we expected it to be a problem at some point. But if we're going to build in complicated hover popups, then uniting them into a shared componet that can function as both (part of a larger SVG) and (a first-class HTML DOM node) will become more complicated, as well.
            Hide
            jamesdumay James Dumay added a comment -

            jmcdonald any alternatives?

            Show
            jamesdumay James Dumay added a comment - jmcdonald any alternatives?
            Hide
            sophistifunk Josh McDonald added a comment -

            No, it's not a super-difficult thing, we just need to separate the standalone component into two, an outer that serves to be the SVG container, and an inner that actually represents the geometry. We'll need to make sure the inner can be used by both the standalone one and as part of the pipeline graph, and also make sure the popup-creation code can be used from both "container" components.

            Fiddly, but nothing fundamentally challenging

            Show
            sophistifunk Josh McDonald added a comment - No, it's not a super-difficult thing, we just need to separate the standalone component into two, an outer that serves to be the SVG container, and an inner that actually represents the geometry. We'll need to make sure the inner can be used by both the standalone one and as part of the pipeline graph, and also make sure the popup-creation code can be used from both "container" components. Fiddly, but nothing fundamentally challenging
            Hide
            jamesdumay James Dumay added a comment -

            jmcdonald sounds reasonable. New ticket for that or should it be included in the scope here?

            Show
            jamesdumay James Dumay added a comment - jmcdonald sounds reasonable. New ticket for that or should it be included in the scope here?
            Hide
            sophistifunk Josh McDonald added a comment -

            I'm not sure what the priority is on this, but I think making the renderers shared should probably be done first, as it'd more pain to do after, and much easier to reason about + review without breaking things if done separately rather than together

            Show
            sophistifunk Josh McDonald added a comment - I'm not sure what the priority is on this, but I think making the renderers shared should probably be done first, as it'd more pain to do after, and much easier to reason about + review without breaking things if done separately rather than together
            jamesdumay James Dumay made changes -
            Link This issue is duplicated by UX-392 [ UX-392 ]
            jamesdumay James Dumay made changes -
            Description In Scope
            * Use the tooltip defined in UX-391
            * Should be done for regular indicators and the indicators in the railroad
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed 2 minutes and 23 seconds and marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Waiting to be run"
            * Unknown – text: "Unknown"

            Not in scope
            * In progress indicator tool tip (covered in UX-392)
            In Scope
            * Use the tooltip defined in UX-391
            * Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Estimated to start in 2 minutes 32 seconds"
            * Unknown – text: "Unknown"
            michaelneale Michael Neale made changes -
            Epic Link UX-79 [ 25277 ] UX-70 [ 25240 ]
            jamesdumay James Dumay made changes -
            Link This issue is blocked by UX-413 [ UX-413 ]
            Hide
            michaelneale Michael Neale added a comment -

            this sounds clever and hard. Is the problem that we want to hover over both SVG things but also dom nodes?

            Show
            michaelneale Michael Neale added a comment - this sounds clever and hard. Is the problem that we want to hover over both SVG things but also dom nodes?
            Hide
            sophistifunk Josh McDonald added a comment -

            The popups will need to be triggered by JS either way, we just need to put some thought into which container they get added to, and how we determine where to put them, in said container's layout space.

            Show
            sophistifunk Josh McDonald added a comment - The popups will need to be triggered by JS either way, we just need to put some thought into which container they get added to, and how we determine where to put them, in said container's layout space.
            Hide
            michaelneale Michael Neale added a comment -

            jmcdonald right, which can have dire consequences if wrong and it pops off the screen ...

            Show
            michaelneale Michael Neale added a comment - jmcdonald right, which can have dire consequences if wrong and it pops off the screen ...
            jamesdumay James Dumay made changes -
            Assignee Joshua McDonald [ jmcdonald ]
            jamesdumay James Dumay made changes -
            Link This issue is blocked by UX-448 [ UX-448 ]
            jamesdumay James Dumay made changes -
            Description In Scope
            * Use the tooltip defined in UX-391
            * Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Estimated to start in 2 minutes 32 seconds"
            * Unknown – text: "Unknown"
            In Scope
            * Use the tooltip defined in UX-391
            * Hovering over each node shows the following tooltip
            * Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Estimated to start in 2 minutes 32 seconds"
            * Unknown – text: "Unknown"
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Sprint 1.0-m7 [ 61 ]
            Hide
            svanoort Sam Van Oort added a comment -

            Comment on this, to consider: parallel blocks (and stages containing them) may have more than one state, I.E. one branch may fail where another passes. How does one represent mixed states?

            Other point: NotExecuted stages/nodes – this can happen when resuming from a checkpoint (the flownodes are created preceding the checkpoint but marked with NotExecutedNodeAction).

            Implementation point: try/catch blocks can generate steps with nodes marked as failed where the overall FLOW still succeeds (error was caught). See https://issues.jenkins-ci.org/browse/JENKINS-34212

            Hopefully by mentioning these issues now, they will avoid tripping you up later (as they have cropped up in stage view and had or are HAVING to be handled).

            Show
            svanoort Sam Van Oort added a comment - Comment on this, to consider: parallel blocks (and stages containing them) may have more than one state, I.E. one branch may fail where another passes. How does one represent mixed states? Other point: NotExecuted stages/nodes – this can happen when resuming from a checkpoint (the flownodes are created preceding the checkpoint but marked with NotExecutedNodeAction). Implementation point: try/catch blocks can generate steps with nodes marked as failed where the overall FLOW still succeeds (error was caught). See https://issues.jenkins-ci.org/browse/JENKINS-34212 Hopefully by mentioning these issues now, they will avoid tripping you up later (as they have cropped up in stage view and had or are HAVING to be handled).
            Hide
            jamesdumay James Dumay added a comment - - edited

            Sam Van Oort you might want to add this comment to the backend ticket - UX-448.

            Usually we have two tickets (frontend and backend) so its clear who is doing what and keep the communication lines clear between the team members.

            Show
            jamesdumay James Dumay added a comment - - edited Sam Van Oort you might want to add this comment to the backend ticket - UX-448 . Usually we have two tickets (frontend and backend) so its clear who is doing what and keep the communication lines clear between the team members.
            Hide
            svanoort Sam Van Oort added a comment -

            James Dumay Ah, good point, thanks – I'll add there too (both ends need to be aware of this since it touches upon the data model used for communications here)

            Show
            svanoort Sam Van Oort added a comment - James Dumay Ah, good point, thanks – I'll add there too (both ends need to be aware of this since it touches upon the data model used for communications here)
            Hide
            michaelneale Michael Neale added a comment -

            Sam Van Oort there are probably a few tickets some of those things are covered.

            In the case of caught errors that result in overall success - I think that is ok. It may look odd and we may need to tweak how it behaves though.

            For parallel: if a branch fails then the "enclosing" stage should probably be shown as a failure.

            I think there will be many more little tickets to flush out these edge cases vs all at once.

            Show
            michaelneale Michael Neale added a comment - Sam Van Oort there are probably a few tickets some of those things are covered. In the case of caught errors that result in overall success - I think that is ok. It may look odd and we may need to tweak how it behaves though. For parallel: if a branch fails then the "enclosing" stage should probably be shown as a failure. I think there will be many more little tickets to flush out these edge cases vs all at once.
            Hide
            svanoort Sam Van Oort added a comment -

            mneale Yes, a failure on a branch indicates a failure for the stage... this is also true for successes across all branches. The other states are less well-defined though (we should consider providing some sort of "fuzzy" option where there are different values).

            For now, I'd be happy seeing something that covers the 90% of cases but isn't going to fall over totally with the other 10%, but I want to make sure they're noted so they don't come up as a surprise.

            Show
            svanoort Sam Van Oort added a comment - mneale Yes, a failure on a branch indicates a failure for the stage... this is also true for successes across all branches. The other states are less well-defined though (we should consider providing some sort of "fuzzy" option where there are different values). For now, I'd be happy seeing something that covers the 90% of cases but isn't going to fall over totally with the other 10%, but I want to make sure they're noted so they don't come up as a surprise.
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            admin V R made changes -
            Project Import Tue Jun 14 12:50:41 JST 2016 [ 1465876241198 ]
            rtyler R. Tyler Croy made changes -
            Project Import Wed Jun 15 02:53:18 UTC 2016 [ 1465959198925 ]
            jamesdumay James Dumay made changes -
            Labels blueocean-imported
            jamesdumay James Dumay made changes -
            Component/s blueocean-plugin [ 21481 ]
            Key UX-393 JENKINS-35849
            Workflow Product Backlog April 2016 V2 [ 172285 ] JNJira [ 172469 ]
            Project User Experience [ 10600 ] Jenkins [ 10172 ]
            Status To Do [ 10003 ] Open [ 1 ]
            admin V R made changes -
            Assignee Joshua McDonald [ jmcdonald ]
            admin V R made changes -
            Priority Medium [ 3 ] Major [ 10000 ]
            admin V R made changes -
            Reporter James Dumay [ jdumay ] James Dumay [ jamesdumay ]
            admin V R made changes -
            Reporter James Dumay [ jdumay ] James Dumay [ jdumay ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 172469 ] JNJira + In-Review [ 184652 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Sprint 1.0-beta-5 [ 106 ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            michaelneale Michael Neale made changes -
            Sprint atlantic [ 106 ] indian [ 126 ]
            michaelneale Michael Neale made changes -
            Link This issue relates to JENKINS-38883 [ JENKINS-38883 ]
            jamesdumay James Dumay made changes -
            Sprint indian [ 126 ] arctic [ 131 ]
            jamesdumay James Dumay made changes -
            Sprint arctic [ 131 ] arctic 2 [ 136 ]
            jamesdumay James Dumay made changes -
            Sprint arctic 2 [ 136 ] arctic [ 131 ]
            jamesdumay James Dumay made changes -
            Sprint arctic [ 131 ] arctic 2 [ 136 ]
            jamesdumay James Dumay made changes -
            Sprint arctic 2 [ 136 ] arctic [ 131 ]
            jamesdumay James Dumay made changes -
            Description In Scope
            * Use the tooltip defined in UX-391
            * Hovering over each node shows the following tooltip
            * Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            * Successful – text: "Passed in 2 minutes and 23 seconds"
            * Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            * Failed – text: "Failed in 2 minutes and 23 seconds"
            * Queued – text: "Estimated to start in 2 minutes 32 seconds"
            * Unknown – text: "Unknown"
            In Scope
            * Hovering over each node shows the following tooltip
            ** Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            ** Successful – text: "Passed in 2 minutes and 23 seconds"
            ** Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            ** Failed – text: "Failed in 2 minutes and 23 seconds"
            ** Queued – text: "Estimated to start in 2 minutes 32 seconds"
            ** Unknown – text: "Unknown"
            jamesdumay James Dumay made changes -
            Description In Scope
            * Hovering over each node shows the following tooltip
            ** Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            ** Successful – text: "Passed in 2 minutes and 23 seconds"
            ** Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            ** Failed – text: "Failed in 2 minutes and 23 seconds"
            ** Queued – text: "Estimated to start in 2 minutes 32 seconds"
            ** Unknown – text: "Unknown"
            In Scope
            * Hovering over each node shows the following tooltip
            ** Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            ** Successful – text: "Passed in 2 minutes and 23 seconds"
            ** Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            ** Failed – text: "Failed in 2 minutes and 23 seconds"
            ** Not built – text: "Estimated to start in 2 minutes 32 seconds"
            ** Unknown – text: "Unknown"
            jamesdumay James Dumay made changes -
            Sprint arctic [ 131 ] tasman [ 136 ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Sprint tasman [ 136 ] frank [ 156 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Sprint frank [ 156 ] frank 2 [ 161 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Assignee Vivek Pandey [ vivek ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Sprint tethys [ 161 ] kraken [ 166 ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Link This issue is duplicated by JENKINS-40680 [ JENKINS-40680 ]
            jamesdumay James Dumay made changes -
            Labels blueocean-imported adoption blueocean-imported
            jamesdumay James Dumay made changes -
            Sprint kraken [ 166 ] tethys [ 161 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Assignee Vivek Pandey [ vivek ] Thorsten Scherler [ tscherler ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Link This issue is duplicated by JENKINS-40711 [ JENKINS-40711 ]
            jamesdumay James Dumay made changes -
            Description In Scope
            * Hovering over each node shows the following tooltip
            ** Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            ** Successful – text: "Passed in 2 minutes and 23 seconds"
            ** Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            ** Failed – text: "Failed in 2 minutes and 23 seconds"
            ** Not built – text: "Estimated to start in 2 minutes 32 seconds"
            ** Unknown – text: "Unknown"
            In Scope
            * Hovering over each node shows the following tooltip
            ** Running – text: "Running for 2 minutes 32 seconds, finishes in 3 minutes 12 seconds"
            ** Successful – text: "Passed in 2 minutes and 23 seconds"
            ** Unstable – text: "Passed in 2 minutes and 23 seconds but marked as unstable."
            ** Failed – text: "Failed in 2 minutes and 23 seconds"
            ** Not built – text: "Estimated to start in 2 minutes 32 seconds"
            ** Unknown – text: "Unknown"
            * All text must be i18n'able
            michaelneale Michael Neale made changes -
            Sprint tethys [ 161 ]
            michaelneale Michael Neale made changes -
            Summary Developer sees good hover states for all indicator types Developer sees good hover states for all indicator types including ETAs
            michaelneale Michael Neale made changes -
            Summary Developer sees good hover states for all indicator types including ETAs Developer sees ETAs on hover states for all indicator types
            jamesdumay James Dumay made changes -
            Sprint iapetus [ 216 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            Hide
            jamesdumay James Dumay added a comment -

            This ticket has been redefined in JENKINS-40929. Please watch this ticket instead.

            Show
            jamesdumay James Dumay added a comment - This ticket has been redefined in JENKINS-40929 . Please watch this ticket instead.
            jamesdumay James Dumay made changes -
            Link This issue duplicates JENKINS-40929 [ JENKINS-40929 ]
            jamesdumay James Dumay made changes -
            Resolution Duplicate [ 3 ]
            Status Open [ 1 ] Resolved [ 5 ]
            jamesdumay James Dumay made changes -
            Remote Link This issue links to "CloudBees Internal UX-602 (Web Link)" [ 18199 ]

              People

              Assignee:
              tscherler Thorsten Scherler
              Reporter:
              jamesdumay James Dumay
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: