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

Make Mac borderless windows resizable and draggable. #740

Merged

Conversation

pcwalton
Copy link
Contributor

Despite the fact that the style mask contains NSTitledWindowMask, the
title doesn't show up for two reasons: (a) we draw over it; (b) we make
it invisible with a call to -[NSWindow setTitleVisibility:].

Addresses servo/servo#9877 and servo/servo#9878.

r? @paulrouget

@paulrouget
Copy link
Contributor

Addresses servo/servo#9877 and servo/servo#9878.

You mean: servo/servo#9856 and servo/servo#9878

@pcwalton pcwalton force-pushed the mac-resize-drag-borderless-tomaka branch from e532d1f to c4c5123 Compare March 14, 2016 23:44
Despite the fact that the style mask contains `NSTitledWindowMask`, the
title doesn't show up for two reasons: (a) we draw over it; (b) we make
it invisible with a call to `-[NSWindow setTitleVisibility:]`.

Addresses servo/servo#9856 and servo/servo#9878.
Partially addresses servo/servo#9812.
@pcwalton pcwalton force-pushed the mac-resize-drag-borderless-tomaka branch from c4c5123 to 7249529 Compare March 14, 2016 23:45
@paulrouget
Copy link
Contributor

r+

@frewsxcv
Copy link
Contributor

Depends on #741.

@paulrouget
Copy link
Contributor

Depends on #741.

Landed.

NSBorderlessWindowMask as NSUInteger
NSBorderlessWindowMask as NSUInteger |
NSResizableWindowMask as NSUInteger |
NSTitledWindowMask as NSUInteger
Copy link
Contributor

Choose a reason for hiding this comment

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

Alternatively, pull out NSResizableWindowMask and NSTitledWindowMask since they're available in every branch of the conditional now.

@pcwalton
Copy link
Contributor Author

@paulrouget @tomaka Is there something I should do to move this along?

@paulrouget
Copy link
Contributor

I was waiting for Travis.

paulrouget added a commit that referenced this pull request Mar 16, 2016
Make Mac borderless windows resizable and draggable.
@paulrouget paulrouget merged commit 3e9a5e7 into rust-windowing:master Mar 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants