Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(3240): Event Landing View & Job List View - Visually differentiate virtual job from normal job #1229

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

sagar1312
Copy link
Member

@sagar1312 sagar1312 commented Nov 8, 2024

Context

Builds for virtual jobs are not executed in a event. In the UI, such builds are not easily distinguishable from non-virtual job builds.

Objective

  • Event landing view (V1 and V2)
    • Add a new icon “fast-forward“ for the virtual job in the workflow graph
image
  • Job List View (V1 and V2)
    • Use the new icon “fast-forward“ for the virtual job under the first column representing the most recent build status
image
  • Pipeline Options View
    • Display the label “Virtual“ (with gray background) next to the job name to indicate when the job is virtual
image

References

screwdriver-cd/screwdriver#3240

License

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@tkyi
Copy link
Member

tkyi commented Nov 8, 2024

For the fast forward icon, should it be in a circle too for consistency?

@@ -93,6 +157,26 @@ module('Integration | Component | pipeline list job cell', function (hooks) {
assert.dom('.job-name').hasText('b');
});

test('it renders a failed build for virtual job', async function (assert) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would this show up as a red fastforward icon?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

@sagar1312 sagar1312 force-pushed the sagar1312-feat-3240-virtual_jobs branch from 778a85f to b917191 Compare November 8, 2024 17:47
@sagar1312
Copy link
Member Author

sagar1312 commented Nov 8, 2024

For the fast forward icon, should it be in a circle too for consistency?

Initially looked for an fast-forward icon inside a circle. We don't have one with the circle border in the font library we are using https://fontawesome.com/v4/icons/

Also when the build is frozen, we use snowflake icon which is not enclosed with in a circle

@tkyi
Copy link
Member

tkyi commented Nov 8, 2024

Hm in the workflowGraph in the main area and in the events list it looks enclosed, but it's not for the overall status in the events list
Screenshot 2024-11-08 at 10 59 06 AM
Screenshot 2024-11-08 at 10 59 15 AM

@sagar1312
Copy link
Member Author

Hm in the workflowGraph in the main area and in the events list it looks enclosed, but it's not for the overall status in the events list Screenshot 2024-11-08 at 10 59 06 AM Screenshot 2024-11-08 at 10 59 15 AM

But, we don't have fast-forward icon inside a circle. For now, we can go with the one with out the circle border.

@sagar1312 sagar1312 force-pushed the sagar1312-feat-3240-virtual_jobs branch from b917191 to c3420b1 Compare November 11, 2024 16:14
@tkyi tkyi merged commit 54de3ab into master Nov 12, 2024
2 checks passed
@tkyi tkyi deleted the sagar1312-feat-3240-virtual_jobs branch November 12, 2024 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants