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

Horizontal stripes on title bar when showing about dialog #9775

Closed
1 task done
warpdesign opened this issue Mar 25, 2024 · 2 comments · Fixed by #9812
Closed
1 task done

Horizontal stripes on title bar when showing about dialog #9775

warpdesign opened this issue Mar 25, 2024 · 2 comments · Fixed by #9812
Labels
defect [core label] workspace Feedback for workspace management, layout, interactions, etc

Comments

@warpdesign
Copy link

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

Click on Zed Preview -> About to show the about dialog.

Happens in both stable & preview releases.

Environment

Zed: v0.128.1 (Zed Preview)
OS: macOS 14.3.1
Memory: 32 GiB
Architecture: aarch64

If applicable, add mockups / screenshots to help explain present your vision of the feature

about_overlay_horizontal-stripes.mov

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

No response

@warpdesign warpdesign added admin read Pending admin review defect [core label] triage Maintainer needs to classify the issue labels Mar 25, 2024
@osiewicz
Copy link
Contributor

osiewicz commented Mar 26, 2024

This happens for any prompt (e.g. for save prompts as well). I've found that it does not happen in fullscreen; with a bit of digging, it turned out that it is somehow related to us moving the traffic lights;
With this unsafe block gone, we no longer have these stripes show up with prompts:

unsafe {
let close_button: id = msg_send![
self.native_window,
standardWindowButton: NSWindowButton::NSWindowCloseButton
];
let min_button: id = msg_send![
self.native_window,
standardWindowButton: NSWindowButton::NSWindowMiniaturizeButton
];
let zoom_button: id = msg_send![
self.native_window,
standardWindowButton: NSWindowButton::NSWindowZoomButton
];
let mut close_button_frame: CGRect = msg_send![close_button, frame];
let mut min_button_frame: CGRect = msg_send![min_button, frame];
let mut zoom_button_frame: CGRect = msg_send![zoom_button, frame];
let mut origin = point(
traffic_light_position.x,
titlebar_height
- traffic_light_position.y
- px(close_button_frame.size.height as f32),
);
let button_spacing =
px((min_button_frame.origin.x - close_button_frame.origin.x) as f32);
close_button_frame.origin = CGPoint::new(origin.x.into(), origin.y.into());
let _: () = msg_send![close_button, setFrame: close_button_frame];
origin.x += button_spacing;
min_button_frame.origin = CGPoint::new(origin.x.into(), origin.y.into());
let _: () = msg_send![min_button, setFrame: min_button_frame];
origin.x += button_spacing;
zoom_button_frame.origin = CGPoint::new(origin.x.into(), origin.y.into());
let _: () = msg_send![zoom_button, setFrame: zoom_button_frame];
origin.x += button_spacing;
}

@CurbaiCode
Copy link

I think this is a duplicate of #9588.

@JosephTLyons JosephTLyons added workspace Feedback for workspace management, layout, interactions, etc and removed triage Maintainer needs to classify the issue admin read Pending admin review labels Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect [core label] workspace Feedback for workspace management, layout, interactions, etc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants