Skip to content

Conversation

OskarKruger
Copy link
Contributor

Description

Changed description for profile button label, as "open/close profile" was misleading since interacting with this button won't close the menu as it appears as an overlay.
Changed the object name also to ensure clarity.

Copy link
Member

@leekelleher leekelleher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@OskarKruger Thanks for tackling this one, the "open/close" label has been confusing for many! I've left a few comments...

@OskarKruger
Copy link
Contributor Author

Hi @leekelleher
Thanks for the feedback!

I made some corrections with your notes.
Reverted the XML configs to original, changed the localization key to simplify it even more into just profileOptions, and updated the call.
Since this is only done for major releases, should this be a candidate for v17 then?

@leekelleher
Copy link
Member

Thanks @OskarKruger! 💪

Since this is only done for major releases, should this be a candidate for v17 then?

We've good to add in new localization keys in the minor releases, so this is fine for 16.3.

The removal should be inline with our deprecation policy, which is "mark as deprecated, then remove the version after next", which would be v18 in this case. (This is to give 3rd-party extension developers time to update their code). Although, this might feel overkill for a localization key. 😬 So I wouldn't worry too much about removing the unused localization keys yet, (we'll tackle that closer to v18).

@OskarKruger
Copy link
Contributor Author

Cheers @leekelleher!
That makes sense to me also to respect that policy. Let me know what changes could be made from here to stay in line with naming conventions etc :)

@leekelleher
Copy link
Member

@OskarKruger If you revert the removal of the openCloseBackofficeProfileOptions localization keys, then this PR is good to merge in! 🚀

Copy link
Member

@leekelleher leekelleher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thank you @OskarKruger! 🚀

@leekelleher leekelleher enabled auto-merge (squash) September 17, 2025 10:21
@leekelleher leekelleher changed the title Misleading profile button label Profile: Corrected misleading button label Sep 17, 2025
@leekelleher leekelleher merged commit 7483f20 into umbraco:main Sep 17, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants