Skip to content

feat: enable secondary label for macOS menu#46887

Merged
jkleinsc merged 4 commits intomainfrom
enable-secondary-label
May 9, 2025
Merged

feat: enable secondary label for macOS menu#46887
jkleinsc merged 4 commits intomainfrom
enable-secondary-label

Conversation

@mlaurencin
Copy link
Copy Markdown
Member

@mlaurencin mlaurencin commented May 1, 2025

Description of Change

Closes #34621

Adds sublabel functionality for macOS >= 14.4.

Tests and docs updates to come

Checklist

Release Notes

Notes: added sublabel functionality for menus on macOS >= 14.4

@deepak1556 @VerteDinde

@mlaurencin mlaurencin added target/35-x-y PR should also be added to the "35-x-y" branch. target/36-x-y PR should also be added to the "36-x-y" branch. target/37-x-y PR should also be added to the "37-x-y" branch. labels May 1, 2025
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label May 1, 2025
@deepak1556 deepak1556 added the semver/minor backwards-compatible functionality label May 2, 2025
mlaurencin and others added 2 commits May 2, 2025 11:29
Copy link
Copy Markdown
Member

@ckerr ckerr left a comment

Choose a reason for hiding this comment

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

API LGTM

@deepak1556
Copy link
Copy Markdown
Member

Should document in https://github.com/electron/electron/blob/main/docs/api/menu-item.md#new-menuitemoptions that sublabel is only supported on macOS >= 14.4

@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label May 8, 2025
Copy link
Copy Markdown
Member

@codebytere codebytere left a comment

Choose a reason for hiding this comment

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

API LGTM

@jkleinsc jkleinsc merged commit 10e4f9a into main May 9, 2025
67 checks passed
@jkleinsc jkleinsc deleted the enable-secondary-label branch May 9, 2025 16:05
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented May 9, 2025

Release Notes Persisted

added sublabel functionality for menus on macOS >= 14.4

@trop
Copy link
Copy Markdown
Contributor

trop bot commented May 9, 2025

I have automatically backported this PR to "36-x-y", please check out #47040

@trop trop bot added in-flight/36-x-y and removed target/36-x-y PR should also be added to the "36-x-y" branch. labels May 9, 2025
@trop
Copy link
Copy Markdown
Contributor

trop bot commented May 9, 2025

I have automatically backported this PR to "35-x-y", please check out #47041

@trop
Copy link
Copy Markdown
Contributor

trop bot commented May 9, 2025

I have automatically backported this PR to "37-x-y", please check out #47042

@trop trop bot added in-flight/35-x-y in-flight/37-x-y merged/37-x-y PR was merged to the "37-x-y" branch. merged/35-x-y PR was merged to the "35-x-y" branch. merged/36-x-y PR was merged to the "36-x-y" branch. and removed target/35-x-y PR should also be added to the "35-x-y" branch. target/37-x-y PR should also be added to the "37-x-y" branch. in-flight/37-x-y in-flight/35-x-y in-flight/36-x-y labels May 9, 2025
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
* feat: enable secondary label for macOS menu

* Update shell/browser/ui/cocoa/electron_menu_controller.mm

Co-authored-by: Robo <hop2deep@gmail.com>

* fix for lint

* update docs for sublabel

---------

Co-authored-by: Robo <hop2deep@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-review/approved ✅ merged/35-x-y PR was merged to the "35-x-y" branch. merged/36-x-y PR was merged to the "36-x-y" branch. merged/37-x-y PR was merged to the "37-x-y" branch. semver/minor backwards-compatible functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: sublabel is not visible for menu items on mac

5 participants