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

Layout control in the wrong place for RTL macOS #164397

Closed
tom-programming opened this issue Oct 23, 2022 · 7 comments · Fixed by #167060 or #174107
Closed

Layout control in the wrong place for RTL macOS #164397

tom-programming opened this issue Oct 23, 2022 · 7 comments · Fixed by #167060 or #174107
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug confirmed Issue has been confirmed by VS Code Team member insiders-released Patch has been released in VS Code Insiders macos Issues with VS Code on MAC/OS X titlebar VS Code main title bar issues verified Verification succeeded
Milestone

Comments

@tom-programming
Copy link

Version of VS Code: 1.72.2

Your operating system: macOS 10.15.7

List of extensions that you have installed: none

Reproducible steps (1... 2... 3...) that cause the issue:

Set the OS language to Hebrew or Arabic. Then open VSCode

What you expected to see, versus what you actually saw

I expect to see the layout control buttons in the left side when the window is RTL, because in the right side there are the close/minimize/resize OS buttons.

Instead they are in the right side and the are basically impossible to click because they are covered with the window buttons.

See image:

problem occuring

@chrmarti chrmarti added l10n-platform Localization platform issues (not wrong translations) and removed triage-needed labels Oct 25, 2022
@chrmarti chrmarti assigned TylerLeonhardt and unassigned chrmarti Oct 25, 2022
@rzhao271
Copy link
Contributor

Confirmed on macOS Monterey with the titlebar style set to "custom" and the system language set to Arabic.

@rzhao271 rzhao271 added bug Issue identified by VS Code Team member as probable bug macos Issues with VS Code on MAC/OS X titlebar VS Code main title bar issues confirmed Issue has been confirmed by VS Code Team member and removed l10n-platform Localization platform issues (not wrong translations) labels Oct 26, 2022
@rzhao271 rzhao271 added this to the November 2022 milestone Oct 26, 2022
@rzhao271
Copy link
Contributor

rzhao271 commented Nov 22, 2022

Related: #161218

rzhao271 added a commit that referenced this issue Nov 23, 2022
rzhao271 added a commit that referenced this issue Nov 23, 2022
@vscodenpa vscodenpa added the unreleased Patch has not yet been released in VS Code Insiders label Nov 23, 2022
@rzhao271
Copy link
Contributor

rzhao271 commented Nov 23, 2022

Verification steps:

  1. On macOS, switch the most preferred system language to a right-to-left language such as Arabic.
  2. On VS Code Insiders, set the titlebar style to custom in the settings, reloading the program if prompted.
  3. The traffic lights should still be on the left side.

@vscodenpa vscodenpa added insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Nov 24, 2022
@lramos15 lramos15 added the verified Verification succeeded label Dec 1, 2022
@lramos15
Copy link
Member

lramos15 commented Dec 1, 2022

@rzhao271 verified this, but isn't it expected that the traffic lights and WCO actually switch so that traffic lights are on the right and the WCO is on the left when using a RTL language?

@rzhao271
Copy link
Contributor

rzhao271 commented Dec 1, 2022

Reopening this issue because the current fix is even less usable, i.e. #167543

@rzhao271
Copy link
Contributor

rzhao271 commented Dec 6, 2022

@sbatten and I looked into other programs on macOS, and it seems that all non-Electron-based programs have the traffic lights on the right side when the system language is Arabic.

An invalid assumption I had was that because Windows set the location of the window controls overlay (WCO) depending on the application language, that macOS would do something similar. Instead, we should just move over the layout controls, as was originally asked.

Not a duplicate of #161218

@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Feb 11, 2023
@sbatten
Copy link
Member

sbatten commented Feb 17, 2023

Verification Steps:

  1. On macOS, set your OS language to an RTL language such as Arabic.
  2. Restart the machine
  3. Start VS Code, traffic controls should be on the right side
  4. Layout controls in the custom title bar should show to the left of the controls

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug confirmed Issue has been confirmed by VS Code Team member insiders-released Patch has been released in VS Code Insiders macos Issues with VS Code on MAC/OS X titlebar VS Code main title bar issues verified Verification succeeded
Projects
Archived in project
8 participants