Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request]: UserMenu - allow to hide "Edit Avatar" button #10599

Closed
1 task done
MarcusNotheis opened this issue Jan 21, 2025 · 3 comments · Fixed by #10619
Closed
1 task done

[Feature Request]: UserMenu - allow to hide "Edit Avatar" button #10599

MarcusNotheis opened this issue Jan 21, 2025 · 3 comments · Fixed by #10619

Comments

@MarcusNotheis
Copy link
Collaborator

Feature Request Description

When using the UserMenu with the UserMenuAccount, there is always an edit avatar icon displayed:

Image

In my opinion, there should be an option to hide this edit button as not every application might have the option to edit the users avatar or name, e.g. if they come from a central IDP.

Proposed Solution

Add a attribute hide-edit-avatar-button (or any better name 🙂 ) to the UserMenuAccount component.

Proposed Alternatives

Offer a CSS part for the edit button

Organization

SAP SIX Technologies / UI5 Web Components for React

Additional Context

No response

Priority

Medium

Privacy Policy

  • I’m not disclosing any internal or sensitive information.
@ilhan007
Copy link
Member

ilhan007 commented Jan 22, 2025

Hello @adrian-bobev could you evaluate this Feature Request

@adrian-bobev
Copy link
Member

Hello Markus,

thanks for the feedback. We are already working on providing such an API requested by another stakeholders.

Best Regards,
Adrian

@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This issue has been resolved in version v2.7.0-rc.2 🎉

The release is available on v2.7.0-rc.2

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants