New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 6 users

Issue metadata

Status: Available
Owner: ----
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Compat

issue 463348

Sign in to add a comment

Issue 296863: Make unicode-bidi:isolate the default for block elements instead of unicode-bidi:embed

Reported by, Sep 23 2013

Issue description

Migrated from WebKit Bugzilla:

Description From Aharon (Vladimir) Lanin 2011-08-03 06:33:08 PST
When a block element is set to display:inline, it is desirable for it to behave bidi-wise as it would if it were display:inline-block. Under CSS2.1, there was no way to accomplish this, and unicode-bidi:embed was used for lack of a better choice. Now that we have unicode-bidi:isolate, it should be the default instead, as the HTML5 spec ( says it should be.

Comment #1 From Aharon (Vladimir) Lanin 2011-08-03 06:57:07 PST
Created attachment
test case

Comment #2 From Aharon (Vladimir) Lanin 2011-08-03 07:08:04 PST
Created attachment
test case

Comment #3 From Eric Seidel 2011-08-08 09:24:20 PST
This is going to make the bidi-isolate code even more performance-critical than it already is. :(  Oh well.

Comment #4 From Aharon (Vladimir) Lanin 2011-08-09 04:50:02 PST
Please note that unicode-bidi:isolate should not have any effect (and thus can be completely ignored) on elements that already constitute separate bidi paragraphs, e.g. elements with display other than inline, or that have float, or that have absolute position, etc. The default is meant for the case when a "block" element like div gets assigned display:inline. This should be a fairly rare case.

Comment #5 From Eric Seidel 2011-09-29 11:45:42 PST
I suspect this may be as simple as adding the CSS from the HTML5 spec into html.css in WebCore:

The important piece is of course the LayoutTests which get landed with any change.

Comment #6 From Aharon (Vladimir) Lanin 2011-11-17 02:13:55 PST
The change needs to be effective for "block" elements whether or not they have a dir attribute. Rules like
  div {display:block; unicode-bidi:isolate;}
are not sufficient because the generic [dir] {unicode-bidi:embed;} is considered more specific and overrides them for elements with a dir attribute. See

Comment 1 by, Sep 24 2013

 Issue 108016  has been merged into this issue.

Comment 2 by, Sep 24 2013


Comment 3 by, Sep 24 2013


Comment 4 by, Sep 24 2013

It'll be super important to have performance test coverage. As Eric said, where we stand now, this will be a performance regression, but I'm not sure how much, and don't currently have a metric to burn down. I'm happy to code up something to look into the implications, since we can test this now.

Comment 5 by, Sep 25 2013

If it really does cause a performance regression, there is something wrong in the implementation, since unicode-bidi:isolate (just like unicode-bidi:embed) is not supposed to have any effect on an element that is not inline.

Comment 6 by, Sep 25 2013

I'm sure there are many things wrong in the implementation. :)  That said, I think we shouldn't be scared, and we should just throw this CSS in html.css and see what (if anything) breaks.  BlinkON is distracting folks today.

Comment 7 by, Sep 26 2013

Aharon is right, of course. I was definitely distracted.

We should just turn this on.

Comment 8 by, Nov 13 2013

Igor: any reason you can think of we shouldn't just post a patch to html.css to turn this on?  I guess I'll do that tomorrow morning since it should be trivial.

Comment 9 by, Nov 13 2013

No, it is OK.

Comment 10 by, Dec 18 2013

Comment 11 by, Jun 18 2014

Lets see if someone can pick this up.

Comment 12 by, Jan 9 2015

Labels: -Cr-Blink-Rendering Cr-Blink-Layout
Migrate from Cr-Blink-Rendering to Cr-Blink-Layout

Comment 13 by, Jun 13 2015


Comment 14 by, Jun 13 2015

Attempting to remove from cc.

Comment 15 by, Jun 15 2015


Comment 16 by, Nov 6 2015

Blocking: chromium:463348

Comment 17 by, Nov 6 2015


Comment 18 by, Nov 6 2015

There is another WIP for this on and from tests IMO it looks promising.

Comment 19 by, Dec 7 2015

Status: Available

Comment 20 by, Dec 8 2015

Status: Assigned

Comment 21 by, May 23 2016

Components: -Blink>RTL
Labels: RTL
Removing Blink>RTL.

Comment 22 by, Aug 1 2017

Is this still relevant?

Comment 23 by, Aug 1 2017

Labels: -Pri-2 Pri-3
Yeah, this is about when author sets 'display: inline' to block elements such as <p>, they automatically create isolation. We implemented the "dir" attribute to isolate, but not this one.

Comment 24 by, Aug 2 2017

I believe still can be revived with not so huge effort.

Comment 25 by, Aug 2 2017

Oh, I wasn't aware there's a WIP, thank you as always to make our bidi support better.

If you can revive it, that's great and I'm happy to review. Otherwise I might try to find some time, but my backlog is rather long currently.

Comment 26 by, Sep 5 2017

Labels: -Type-Bug Type-Feature
Owner: ----
Status: Available (was: Assigned)

Comment 27 by, Feb 26 2018

 Issue 816626  has been merged into this issue.

Comment 28 by, Feb 27 2018

I still believe can be revived if it could find an owner.

Comment 29 by, Feb 27 2018

Do you mean a reviewer, or someone to upload the patch?

Comment 30 by, Feb 28 2018

Labels: -Type-Feature Type-Compat
Someone to re-upload the patch and fixes minor issue (my experience on last time) it may causes on tests.

Sign in to add a comment