Skip to content

Captions RTL languages such as hebrew + arabic need to be right aligned#2999

Merged
carocao-msft merged 12 commits intomainfrom
carocao/rtl
May 3, 2023
Merged

Captions RTL languages such as hebrew + arabic need to be right aligned#2999
carocao-msft merged 12 commits intomainfrom
carocao/rtl

Conversation

@carocao-msft
Copy link
Copy Markdown
Contributor

@carocao-msft carocao-msft commented Apr 28, 2023

What

Captions RTL languages such as hebrew + arabic need to be right aligned

Why

https://skype.visualstudio.com/SPOOL/_workitems/edit/3233581

How Tested

Screenshot 2023-04-28 at 2 48 31 PM

Screenshot 2023-04-28 at 2 48 24 PM

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2023

CallWithChat bundle size is increased❗.

  • Current size: 10192446
  • Base size: 10192445
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2023

Chat bundle size is increased❗.

  • Current size: 9839822
  • Base size: 9839807
  • Diff size: 15

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2023

Calling bundle size is increased❗.

  • Current size: 9799485
  • Base size: 9799484
  • Diff size: 1

@JamesBurnside
Copy link
Copy Markdown
Member

Can we make an e2e test for this?

@github-actions
Copy link
Copy Markdown
Contributor

@carocao-msft carocao-msft added the update_snapshots Set this label to request automated update of UI snapshots label Apr 28, 2023
@github-actions github-actions Bot removed the update_snapshots Set this label to request automated update of UI snapshots label Apr 28, 2023
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2023

Copy link
Copy Markdown
Contributor

@PorterNan PorterNan left a comment

Choose a reason for hiding this comment

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

Nice to see more hermetic tests on captions!

@carocao-msft carocao-msft enabled auto-merge (squash) May 3, 2023 19:55
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 3, 2023

@carocao-msft carocao-msft merged commit 48830ac into main May 3, 2023
@carocao-msft carocao-msft deleted the carocao/rtl branch May 3, 2023 20:17
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.

4 participants