Garrett LeSage

Results 776 comments of Garrett LeSage

We actually do have a confirmation dialog on these danger/destructive states within Cockpit. If there's a destructive action that can be undone with an undo action, then having a danger...

@mcarrano: This feature doesn't have anything to do with avoiding confirmation dialogs. It's just marking dangerous actions with a visual style. A dangerous action would still need to have either...

Apologies, but I can't see any of those mockups. First mockup link: ![image](https://user-images.githubusercontent.com/10246/182887009-a5ae6ee7-39b0-473a-9d28-1ac3e78a25cf.png) Second mockup link: ![image](https://user-images.githubusercontent.com/10246/182887092-73c518cc-b72b-4178-b207-00c99498f7a6.png)

Thanks! That's a lot better. Here's an inline image of both options: ![image](https://user-images.githubusercontent.com/10246/183448598-f70b272a-1d72-4aea-9526-23276db54e33.png) I think since it is a destructive action, it does make sense to highlight that — therefore...

Firstly, reminder: This mockup was made as a response to a user study. PatternFly currently does not solve the issue in the study, so the installer team will either have...

I've implemented the mockup in basically 5 lines of CSS (mainly to set colors and a width for percentage), where the percentage is exposed as a number (representing a percentage),...

I've made a modified second demo which has closer-to-ready SCSS (tying it to the widget), adding it as a pf-m- variant, uses success and current colors, and adds a transition:...

> In your proposed design above (afaik) the bar is calculating progress between steps 1-2, 2-3, rather than overall progress. No, I am actually talking about the current progress for...

> By "between" I meant Step x has been completed, now Step next is in progress. We might be talking about the same thing? Yeah, in the demo I made,...

> If coloring the step circle and segment line is already possible with the current component, maybe that's something that might be feasible to consider in the short term? It...