Skip to content

fix: Lower zindex of User Card so it appears below dialogs#1029

Open
Pecacheu wants to merge 1 commit intostoatchat:mainfrom
Pecacheu:cardz
Open

fix: Lower zindex of User Card so it appears below dialogs#1029
Pecacheu wants to merge 1 commit intostoatchat:mainfrom
Pecacheu:cardz

Conversation

@Pecacheu
Copy link
Copy Markdown
Contributor

Fixes #1028

Before & After

@gabrielfordevelopment
Copy link
Copy Markdown
Contributor

I checked it out and it is inconsistent in the current system, since you have two options to go to the "give role" modal:

(You didn't touched this, just saying that it exist)
Click on profile → click on "Roles" - then it opens up the role editor and closes all other components.
image

(This is what you changed)
However, if you do Profile → 3 dots → Edit Roles it does NOT close the other components
image

This is inconsistent. Consider: close all components here too instead of adjusting the Z index to be in front.

@Pecacheu
Copy link
Copy Markdown
Contributor Author

This is inconsistent. Consider: close all components here too instead of adjusting the Z index to be in front.

Ah, I didn't even realize you could click that to edit roles lol

You're right, it's two different behaviors when both options should act in the same way. I can change it to dismiss the User Card instead of alter the z-index, but now, since both behaviors exist in the current build, the question becomes... which one is the intended way to handle it from a UI perspective... Roles open as a separate sub-menu, or roles open as a new overlay that replaces the old one?

Probably a question for the UI/UX team.

@Pecacheu Pecacheu added the ux review This needs UX review label Mar 15, 2026
@ThomasJRyan ThomasJRyan requested review from a team March 16, 2026 03:17
Fixes stoatchat#1028

Signed-off-by: Pecacheu <3608878+Pecacheu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ux review This needs UX review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Edit Roles menu appears below User Card in hierarchy

2 participants