Skip to content

Commit

Permalink
fix RTL bug when used with traffic light repositioning (#22162)
Browse files Browse the repository at this point in the history
Co-authored-by: tonyfwoo <55114329+tonyfwoo@users.noreply.github.com>
  • Loading branch information
trop[bot] and tonyfwoo committed Feb 12, 2020
1 parent dc9654c commit e9814e0
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion shell/browser/native_window_mac.mm
Expand Up @@ -546,17 +546,27 @@ void ViewDidMoveToSuperview(NSView* self, SEL _cmd) {
CGFloat buttonHeight = [close frame].size.height;
CGFloat titleBarFrameHeight = buttonHeight + traffic_light_position_.y();
CGRect titleBarRect = titleBarContainerView.frame;
CGFloat titleBarWidth = NSWidth(titleBarRect);
titleBarRect.size.height = titleBarFrameHeight;
titleBarRect.origin.y = window.frame.size.height - titleBarFrameHeight;
[titleBarContainerView setFrame:titleBarRect];

BOOL isRTL = [titleBarContainerView userInterfaceLayoutDirection] ==
NSUserInterfaceLayoutDirectionRightToLeft;
NSArray* windowButtons = @[ close, miniaturize, zoom ];
const CGFloat space_between =
[miniaturize frame].origin.x - [close frame].origin.x;
for (NSUInteger i = 0; i < windowButtons.count; i++) {
NSView* view = [windowButtons objectAtIndex:i];
CGRect rect = [view frame];
rect.origin.x = traffic_light_position_.x() + (i * space_between);
if (isRTL) {
CGFloat buttonWidth = NSWidth(rect);
// origin is always top-left, even in RTL
rect.origin.x = titleBarWidth - traffic_light_position_.x() +
(i * space_between) - buttonWidth;
} else {
rect.origin.x = traffic_light_position_.x() + (i * space_between);
}
rect.origin.y = (titleBarFrameHeight - rect.size.height) / 2;
[view setFrameOrigin:rect.origin];
}
Expand Down

0 comments on commit e9814e0

Please sign in to comment.