Feedback to new two-row pipeline layout

[pat-s]

Component

web-ui

Describe the bug

@qwerty287

• Min-width of commit hash field should be larger
• Sometimes the committer icon is missing (maybe use an anchored relative position for the start of the timing buttons so it doesn't matter if the icon is missing?)
• Maybe the committer icon could also be moved as the first entry of the second line? This would avoid the layout shift in case it is missing (although it should always exist actually, not sure why it is missing in some runs in the new view)

[pat-s]

Additional issues:

• The status indicator was changed from w-3 to w-2 and now looks way to thin. Was this in purpose, @qwerty287?
• Icons are not aligned with the text anymore

@qwerty287 Should we maybe revert and have a new clean PR?

[qwerty287]

Should we maybe revert and have a new clean PR?

Maybe yes.

Tbh I just took the changes from @anbraten and added icons as requested in the original pr.

I mainly wanted to get the pipeline message and title/context (e.g. commit and pr title) difference what was done there...

[pat-s]

All good, I also didn't test it in detail and I am partly guilty for merging :)

Let's revert and test it in more detail with PR images? Do you wanna revert or should I?

[qwerty287]

You can do it, thanks

[xoxys]

Maybe the committer icon could also be moved as the first entry of the second line? This would avoid the layout shift in case it is missing (although it should always exist actually, not sure why it is missing in some runs in the new view)

Its also possible to just reserve the required space (which is a good practice for images anyway)

[pat-s]

reverted via #4649, new discussions should happen in a new PR then.