Skip to content

Move Application:: hide_application, hide_others and set_menu to Mac platform extension#1863

Merged
maan2003 merged 5 commits intolinebender:masterfrom
maan2003:mac-ext
Jul 8, 2021
Merged

Move Application:: hide_application, hide_others and set_menu to Mac platform extension#1863
maan2003 merged 5 commits intolinebender:masterfrom
maan2003:mac-ext

Conversation

@maan2003
Copy link
Collaborator

@maan2003 maan2003 commented Jul 7, 2021

No description provided.

Copy link
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

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

Love to see this, thanks 🥳


/// Hide the application. (mac only?)
/// Hide the application. (mac only)
#[cfg_attr(
Copy link
Member

Choose a reason for hiding this comment

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

I spent a few minutes thinking about whether this is too fancy, but I think it's a good change. 👍

@maan2003
Copy link
Collaborator Author

maan2003 commented Jul 7, 2021

why are menu::sys::{os} not behind cfg?

@cmyr
Copy link
Member

cmyr commented Jul 7, 2021

why are menu::sys::{os} not behind cfg?

not for any concrete reason.

@maan2003
Copy link
Collaborator Author

maan2003 commented Jul 7, 2021

lets leave it for now and allow(deprecated). I would prefer it to be separate PR, if needed

@maan2003 maan2003 merged commit 52f257b into linebender:master Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants