drogue-iot / drogue-cloud

Cloud Native IoT
https://drogue.io
Apache License 2.0
113 stars 30 forks source link

Console: Allow adding description when creating access keys #328

Closed jbtrystram closed 1 year ago

jbtrystram commented 1 year ago

When clicking on the create button:

image

Then a confirmation modal : image

jbtrystram commented 1 year ago

@ctron I think that requires some change in the integration tests ?

ctron commented 1 year ago

@ctron I think that requires some change in the integration tests ?

Yea, that could be.

ctron commented 1 year ago

This is great! There are just two small things I think we could improve:

Add a bit of padding between the copy & paste field and the top/bottom text.

Second, I am unsure about the yellow button. The outcome is "success", but the button indicates "warning", but doesn't highlight the "why".

Here is an example from Azure:

image

I also saw (can't find it now) a checkbox (with "I copied the token and know that I can't get it back") plus disabling the close button as long as the checkbox is not ticked.

jbtrystram commented 1 year ago

I also saw (can't find it now) a checkbox (with "I copied the token and know that I can't get it back") plus disabling the close button as long as the checkbox is not ticked.

I don't like this, it feels like something frustrating that i would not really read :D The yellow box with a warning that you showed is good imho, it attract the eyes and get the the point across.

ctron commented 1 year ago

IIRC Patternfly had an "inline" variant for this: https://www.patternfly.org/v4/components/alert/#inline-types

Not sure we support it yet.

jbtrystram commented 1 year ago

@ctron here is the redesigned modal : image

Thanks for the feedback !