ondemand icon indicating copy to clipboard operation
ondemand copied to clipboard

Re-Design Navbar in regards to "Logged in as...", "Help" & Log Out

Open gerald-byrket opened this issue 2 years ago • 6 comments

We should change the "Logged in as" section to something like this.

Top Menu = "user's name"

image

From the image above move the following to be under the Top Menu of the "User's Name". Not necessarily in the order listed. image then this.

  • Change HPC Password
  • Manage Federated ID
  • Log Out

And remove "Log Out" from the Top Level of the Navbar.

Originally posted by @gerald-byrket in https://github.com/OSC/ondemand/issues/2484#issuecomment-1382138218

┆Issue is synchronized with this Asana task by Unito

gerald-byrket avatar Jan 13 '23 18:01 gerald-byrket

Your image from the other PR. This looks great.

Screenshot 2023-01-13 at 20 02 54

gerald-byrket avatar Jan 18 '23 18:01 gerald-byrket

@adaybujeda . This is the GH Issue for the menu updates.

gerald-byrket avatar Jan 18 '23 18:01 gerald-byrket

Thanks @gerald-byrket. Will create a new PR just related to these changes.

abujeda avatar Jan 18 '23 18:01 abujeda

IDK if I'm sold on this actually. I'd +1 to remove it altogether, but IDK if having it as a menu item is better than help. help is straight forward to understand what that menu may have in it.

What is in help menu if we move it all to this menu?

johrstrom avatar Jan 18 '23 19:01 johrstrom

I think the image that I generated is misleading. I was meant to show the menu only, not the actual links inside.

I think the links would be:

Help:

  • documentation and help links
  • restart server
  • support ticket

User Menu:

  • Change HPC Password
  • Manage Federated ID
  • Log Out
  • Profile links

abujeda avatar Jan 18 '23 20:01 abujeda

Created PR in case you want to push forward with this change.

https://github.com/OSC/ondemand/pull/2499

abujeda avatar Jan 19 '23 11:01 abujeda