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

Fix Touch Bar time remaining label in RTL, #4946 #4967

Merged
merged 1 commit into from
Jun 1, 2024
Merged

Conversation

low-batt
Copy link
Contributor

@low-batt low-batt commented Jun 1, 2024

This commit will change the touchBar method in TouchBarSupport to set baseWritingDirection to leftToRight on the DurationDisplayTextField that shows time remaining.

This is a follow-up to PR #4958 which changed the baseWritingDirection in the XIB for the OSC. That fix is correct, but does not fix the DurationDisplayTextField created at runtime by TouchBarSupport. This commit corrects the TouchBarSupport use case.


Description:

This commit will change the touchBar method in TouchBarSupport to set
baseWritingDirection to leftToRight on the DurationDisplayTextField that
shows time remaining.

This is a follow-up to PR #4958 which changed the baseWritingDirection
in the XIB for the OSC. That fix is correct, but does not fix the
DurationDisplayTextField created at runtime by  TouchBarSupport. This
commit corrects the TouchBarSupport use case.
@low-batt low-batt requested a review from uiryuu June 1, 2024 04:51
Copy link
Member

@uiryuu uiryuu left a comment

Choose a reason for hiding this comment

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

Pulled and tested.

@uiryuu uiryuu merged commit 9196c56 into develop Jun 1, 2024
1 check passed
@uiryuu uiryuu deleted the touchbar-label branch June 1, 2024 12:46
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