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

[questions/qa-bidi-controls] CSS shim link broken #518

Closed
xfq opened this issue Aug 21, 2023 · 3 comments · Fixed by #532
Closed

[questions/qa-bidi-controls] CSS shim link broken #518

xfq opened this issue Aug 21, 2023 · 3 comments · Fixed by #532

Comments

@xfq
Copy link
Member

xfq commented Aug 21, 2023

[source] (https://www.w3.org/International/questions/qa-bidi-controls#reasons) [en]

Currently, the adoption of those new control codes is lagging in browsers, whereas isolation can be applied already using markup. (At the moment you can use a CSS shim, or dedicated elements such as bdi, but browsers are also working on applying isolation to the use of the dir attribute by default.)

The CSS shim link seems broken.

@aphillips
Copy link
Contributor

The text is also somewhat stale?

@r12a
Copy link
Contributor

r12a commented Aug 22, 2023

I suggest replacing the following:

Another, hopefully temporary, issue at the time of writing is that the Unicode Consortium and the W3C recommend applying isolation to embedded ranges of text where the base direction is changed. This means that you should use the RLI, LRI and PDI control codes, rather than the RLE, LRE and PDF ones. Currently, the adoption of those new control codes is lagging in browsers, whereas isolation can be applied already using markup. (At the moment you can use a CSS shim, or dedicated elements such as bdi, but browsers are also working on applying isolation to the use of the dir attribute by default.)

with

The Unicode Consortium and the W3C recommend applying isolation to embedded ranges of text where the base direction is changed. This means that you should use the RLI, LRI and PDI control codes, rather than the RLE, LRE and PDF ones.

xfq pushed a commit that referenced this issue Sep 7, 2023
@xfq
Copy link
Member Author

xfq commented Sep 7, 2023

Yes, that looks good to me. I'll raise a PR with git commit --author=r12a.

@xfq xfq closed this as completed in #532 Dec 14, 2023
xfq added a commit that referenced this issue Dec 14, 2023
Fix #518.

Co-authored-by: r12a <ishida@w3.org>
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 a pull request may close this issue.

3 participants