Unprefix -moz-text-align-last

RESOLVED FIXED in Firefox 49

Status

()

defect
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: bfrohs, Assigned: xidorn)

Tracking

(Blocks 2 bugs, {dev-doc-complete, site-compat})

30 Branch
mozilla49
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox49 fixed)

Details

(Whiteboard: [parity-blink])

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:30.0) Gecko/20100101 Firefox/30.0 (Beta/Release)
Build ID: 20140608211622

Steps to reproduce:

Added `text-align-last:justify` to an element.


Actual results:

`text-align-last` was discarded. `-moz-text-align-last` works.


Expected results:

`text-align-last` should be accepted like `-moz-text-align-last`.

Spec: http://www.w3.org/TR/css-text-3/#text-align-last-property

Last change to `text-align-last` was in the latest version (10 October 2013).

One issue regarding `text-align-last`: http://www.w3.org/Style/CSS/Tracker/issues/333

Side note: Gmail discards `-moz-text-align-last` in emails, but keeps `text-align-last`.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Updated

3 years ago
Whiteboard: [parity-blink][parity-edge]
(Assignee)

Comment 1

3 years ago
So it seems we pass all tests in CSSWG on text-align-last if we unprefix it.
Could you (1) send an intent to ship and (2) file a bug to remove the alias?

(I think shipping unprefixed seems reasonable if that's what blink and edge have done (which your whiteboard annotation makes me suspect) or if it's in a CR draft (which I'm not sure of).)
(Assignee)

Comment 5

3 years ago
caniuse.com says Edge and even IE does, but it seems it is wrong. According to my test, Blink indeed has shipped unprefixed text-align-last, but Edge doesn't.
Whiteboard: [parity-blink][parity-edge] → [parity-blink]
(Assignee)

Comment 6

3 years ago
Comment on attachment 8757164 [details]
MozReview Request: Bug 1039541 part 1 - Unprefix text-align-last. r?heycam

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/55742/diff/1-2/
(Assignee)

Comment 7

3 years ago
Comment on attachment 8757165 [details]
MozReview Request: Bug 1039541 part 2 - Use unprefixed text-align-last in tests. r?heycam

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/55744/diff/1-2/
(Assignee)

Comment 9

3 years ago
Comment on attachment 8757164 [details]
MozReview Request: Bug 1039541 part 1 - Unprefix text-align-last. r?heycam

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/55742/diff/2-3/
(Assignee)

Comment 10

3 years ago
Comment on attachment 8757165 [details]
MozReview Request: Bug 1039541 part 2 - Use unprefixed text-align-last in tests. r?heycam

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/55744/diff/2-3/
(In reply to Xidorn Quan [:xidorn] (UTC+10) from comment #5)
> caniuse.com says Edge and even IE does, but it seems it is wrong. According
> to my test, Blink indeed has shipped unprefixed text-align-last, but Edge
> doesn't.

Maybe file an issue at https://github.com/Fyrd/caniuse/issues ?
Attachment #8757164 - Flags: review?(cam) → review+
Comment on attachment 8757165 [details]
MozReview Request: Bug 1039541 part 2 - Use unprefixed text-align-last in tests. r?heycam

https://reviewboard.mozilla.org/r/55744/#review52780
Attachment #8757165 - Flags: review?(cam) → review+
(Assignee)

Updated

3 years ago
Assignee: nobody → bugzilla

Comment 15

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/16f29c411ce6
https://hg.mozilla.org/mozilla-central/rev/cec360c7fbc9
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
(Assignee)

Updated

3 years ago
Blocks: 1276808
Thanks percyley!
You need to log in before you can comment on or make changes to this bug.