search

integr8ly / tutorial-web-app

Solution Explorer provides the front door into the Integreatly initiative. It hosts the various Solution Patterns, as well as providing a dashboard of installed applications/products/services.
Apache License 2.0
36 stars 54 forks source link

APPDUX-81: Add role info under user menu #581

Closed mfrances17 closed 4 years ago

mfrances17 commented 4 years ago

Motivation

https://issues.redhat.com/browse/APPDUX-81

What

Add login name, admin/dev icon, and admin/dev 'badge' based on whether the current user has admin rights or not, according to the design spec.

Why

To better display whether the currently logged in user has admin account privileges or not.

Verification Steps

No cluster to test on, so you will need to verify by running locally:

  1. Start solution explorer and click the user menu in the masthead. It should display the administrator version of the menu.
  2. Change ln 230 of masthead.js to 'isAdmin = false'.
  3. Click the user menu in the masthead. It should now display the developer version of the menu.

Checklist:

Progress

Additional Notes

User menu when logged in as Admin: admin_user_menu

User menu when logged in as Dev: dev_user_menu

tiffanynolan commented 4 years ago

Looking good! Just a few styling tweaks.

mfrances17 commented 4 years ago

Looking good! Just a few styling tweaks.

  • No hover state/background color change for the profile info section.
  • Background color for the section with the profile info: global_BackgroundColor_150
  • Background color for the developer label should be: global_palette_cyan_100

Thanks @tiffanynolan... I made the CSS changes. I've attached the latest screen caps, and/or you can run it locally and let me know if you approve.

admin

dev

mfrances17 commented 4 years ago

Looks great! Thanks, Mark.

Thanks... here's a screenie of the correct dev color, just to keep me honest.

correct-dev-color