Refactor and redesign user menu#32812
Conversation
|
I think you mean the user menu? Quick settings is down the bottom. |
|
CI is not happy, is it ready to be reviewed? |
Yeah, I think the design is wrong. I'll ask for it to be changed.
No, I got overager by getting the design done and then failing to check if anything else was happy. Apologies, I'll go clean up. |
|
I've updated the component to be named User Menu |
I don't really understand why the thing it asserts matters, so I'm removing it for now.
I'm not really sure what it meant for this interface to have a property with a default value, so I've removed it and added the property to the view model.
& hopefully make hover consistent in one
|
Home icon is now outline and image is not shown for guests, should be ready for another look. |
Something doesn't feel quite right here... @americanrefugee & @Half-Shot can you please review? Attached are 2 screenshots, one from Desktop and one from Web. The newer version of the menu seems much much larger and as a result very cramped. There's also a bug with overlapping top bar on electron... Are we aligned that these are the correct buttons in the quick action menu? Do you know if we have tracking on the existing buttons? Can we unmerge/resolve before release please? 🙏 |
|
btw @t3chguy that Create an account button looks wrong :/ |
|
Thanks for the response @americanrefugee -- it would appear there are bugs in the implementation then that need resolution... it currently feels far too big (esp. considering the number of options is reduced). I will 'hold space' for this being change aversion but currently I'm not happy for this to be released :/ |
Are you not happy with the overall design, the implementation, or both? |
This is a heading of size md, weight semibold |
|
Let's make the following changes:
|
|
Opened #33444 to make these changes - updated screenshots should give an idea. |
I requested a change on the new issue. Making sure @daniellekirkwood sees it too :) |



Based on https://www.figma.com/design/rTaQE2nIUSLav4Tg3nozq7/Compound-Web-Components?node-id=11583-3362&t=DwFpi7Zlq9uJr1SQ-0
The major new change out of this is that the Sign out button has been removed as per designs, leaving the remaining place to sign out under the Settings -> Profile or Settings > Sessions > Current session page.
Checklist
public/exportedsymbols have accurate TSDoc documentation.