Hixie (not reading bugmail) Reporter
	Description • 22 years ago

A cell whose align attribute is set to "left" overrides the UA stylesheet rule
for <hr> elements in tables that are nested within it.

However, a cell whose align attribute is set to "right" does not.

This is inconsistent.

It is unclear to me which behaviour is technically correct. IE does what we do
for "left" for "right" as well, but a CSS interpretation might suggest there
should be no magic involved, and therefore the align attribute shouldn't affect
nested elements' ua.css rules, and our "right" behaviour is the correct one.

In any case, we should be consistent.

jesus_X said he'd attach a testcase.

	Felix Miata
	Comment 3 • 22 years ago

s/<hr>/<th>/

I discovered this behavior today using OS/2: W2K -> All.

I suggest the better behavior is for neither align=left nor align=right to be
inheritable, unlike IE6, which allows both to inherit. Netscape 4 allows neither
to inherit. 

	Boris Zbarsky [:bzbarsky]
	Comment 4 • 22 years ago

> A cell whose align attribute is set to "left" overrides the UA stylesheet rule
> for <hr> elements in tables that are nested within it.

There is no such rule, actually.  So we just inherit the text-align through; the
only reason "left" and "right" act differently is because align="left" computes
to "text-align:left" and align="right" computes to "text-align: -moz-right" and
then we hit the code in nsStyleContext::ApplyStyleFixups which basically resets
-moz-right and -moz-center to "initial" if they are applied to a table (where
"table" is "display:table").

In quirks mode, text-align is explicitly set to "initial" in quirk.css for
<table>, so none of this is an issue.

So if there were no magic involved we would act exactly like IE does.

Felix, should align="right" inherit into a <p>?  If yes, then why should it not
inherit into a <table>?  (Note that this is all in standards mode; in quirks
mode we already do something consistent.)

	David Baron :dbaron: Assignee
	Comment 5 • 22 years ago

So, in other words, it would be fixed by the patch on bug 64510?

What do other browsers do?

	Boris Zbarsky [:bzbarsky]
	Comment 6 • 22 years ago

It'd be fixed to be consistent and to never inherit HTMl align into a table, yes.

	Boris Zbarsky [:bzbarsky]
	Comment 10 • 22 years ago

> HTML testcase - strict - *maybe* align= should inherit directly into <P>
>(quirks doesn't)

Sure it does.  Remove the doctype -- the align inherits into the <p>.

So the question remains.  In _standards_ mode, should we treat <p> and <table>
differently here?

	Felix Miata
	Comment 11 • 22 years ago

Slap me for sloppy edit after cut & paste. Up too late on IRC last night. Still
not recovered? Let me try again.

Comment 7 (attachment 116542 [details] caption): s/(quirks doesn't)/(quirks does)/

I vote for bug 64510 fix "never inherit HTML align into a table" as per comment
6.

	Allan Crossman
	Comment 12 • 21 years ago

This file shows inconsistent behaviour when the font size is changed while
viewing.

	Bernd
	Comment 13 • 15 years ago

No other browser inherits in standards mode testcase, IE8, webkit and Opera all show both P centered in the standards mode.

	Boris Zbarsky [:bzbarsky]
	Comment 14 • 15 years ago

Does text-align inherit into tables in those browsers by default?

What does the testcase look like in those browsers if you set text-align:inherit on the table?

Or is it just that they don't map <td align=""> to text-align?

	Boris Zbarsky [:bzbarsky]
	Comment 15 • 15 years ago

Note that we could also just introduce align:-moz-left if we wanted...

	David Baron :dbaron: Assignee
	Comment 16 • 9 years ago

(In reply to Boris Zbarsky [:bz] from comment #15)
> Note that we could also just introduce align:-moz-left if we wanted...

We did that in bug 64510.

	David Baron :dbaron: Assignee
	Comment 17 • 9 years ago

... but I didn't add this piece:
https://hg.mozilla.org/users/dbaron_mozilla.com/patches/raw-file/bfa3993c6c16/table-reset-alignment-left

I should test that it's consistent with both Chromium and Edge.

	David Baron :dbaron: Assignee
	Comment 18 • 9 years ago

So Chromium 45 and IE11 match on the 5 testcases on this bug (attachment 116522 [details], attachment 116523 [details], attachment 116542 [details], attachment 116543 [details], attachment 116568 [details]).  I can't test Edge now, although I might have a chance to do so tomorrow.

	David Baron :dbaron: Assignee
	Comment 19 • 9 years ago

... continuing from comment 18:  current Gecko differs on the second and fourth of those testcases (on the second table in them).  With the patch in comment 17, it matches on all 5.

	David Baron :dbaron: Assignee
	Comment 20 • 9 years ago

Without the patch, cell-align-stopped-at-table-1-standards.html fails
because the th in the align=left cell is left-aligned rather than
centered.

	David Baron :dbaron: Assignee
	Comment 21 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=70ed5feb6d47&group_state=expanded

	:Ms2ger (he/him; ⌚ UTC+1/+2)
	Comment 22 • 9 years ago

Would you mind pointing me at the spec here?

	Boris Zbarsky [:bzbarsky]
	Comment 23 • 9 years ago

The spec at <https://html.spec.whatwg.org/multipage/rendering.html#flow-content-3> says:

  The div element, when it has an align attribute whose value is an ASCII case-insensitive
  match for the string "left", is expected to left-align text within itself, as if it had
  its 'text-align' property set to 'left' in a presentational hint, and to align
  descendants to the left.

  The div element, when it has an align attribute whose value is an ASCII case-insensitive
  match for the string "right", is expected to right-align text within itself, as if it
  had its 'text-align' property set to 'right' in a presentational hint, and to align
  descendants to the right.

and similar for "center"/"middle".

Now it's not clear to me what "as if" means in terms of CSS inheritance.  

https://html.spec.whatwg.org/multipage/rendering.html#tables-2 explicitly lists the following as preshints:

  p[align=right i], h1[align=right i], h2[align=right i], h3[align=right i],
  h4[align=right i], h5[align=right i], h6[align=right i] {
    text-align: right;
  }

etc.  But that doesn't apply to <div>, apparently.  And nothing special in there for <table>.

	Boris Zbarsky [:bzbarsky]
	Comment 24 • 9 years ago

Comment on attachment 8693381 [details] [diff] [review]
Make table inside align=left reset alignment just like for align=center and align=right

r=me, but please raise a spec issue on HTML.

	David Baron :dbaron: Assignee
	Comment 25 • 9 years ago

(In reply to Boris Zbarsky [:bz] from comment #23)
> The spec at
> <https://html.spec.whatwg.org/multipage/rendering.html#flow-content-3> says:
> 
>   The div element, when it has an align attribute whose value is an ASCII
> case-insensitive
>   match for the string "left", is expected to left-align text within itself,
> as if it had
>   its 'text-align' property set to 'left' in a presentational hint, and to
> align
>   descendants to the left.

To be clear "align descendants" is a link to something that's actually not that far off from correct:

https://html.spec.whatwg.org/multipage/rendering.html#align-descendants

When a user agent is to align descendants of a node, the user agent is expected to align only those descendants that have both their 'margin-left' and 'margin-right' properties computing to a value other than 'auto', that are over-constrained and that have one of those two margins with a used value forced to a greater value, and that do not themselves have an applicable align attribute. When multiple elements are to align a particular descendant, the most deeply nested such element is expected to override the others. Aligned elements are expected to be aligned by having the used values of their left and right margins be set accordingly.


But I'll raise some issues.

	David Baron :dbaron: Assignee
	Comment 26 • 9 years ago

https://github.com/whatwg/html/issues/369 is the issue that represents the missing bit most strictly related to this bug fix

	David Baron :dbaron: Assignee
	Comment 27 • 9 years ago

Also filed 370.  Also noticed that in our implementation, h1...h6 have the "align descendants" behavior, but that seems like somethnig that might be possible to remove, so I didn't file a bug on it.

	David Baron :dbaron: Assignee
	Comment 28 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/983a6d09ddea6c851c5e2fa1fc1f49a0f0a007fa
Bug 196292 - Make table inside align=left reset alignment just like for align=center and align=right.  r=bz

	Carsten Book [:Tomcat]
	Comment 29 • 9 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/983a6d09ddea