the <canvas> element returns incorrect min/pref inline-size in vertical writing modes

RESOLVED FIXED in mozilla37

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jfkthame, Assigned: jfkthame)

Tracking

(Blocks: 1 bug)

unspecified
mozilla37
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

4 years ago
Created attachment 8532841 [details]
testcase for canvas size in vertical mode

For a simple demonstration, put a <canvas> inside an html <button>, as in the attached testcase, and note how the *width* of the canvas contributes to the *height* of the enclosing button.
(Assignee)

Comment 1

4 years ago
Created attachment 8532842 [details] [diff] [review]
Make Get{Min,Pref}ISize for the <canvas> element aware of vertical writing mode
Attachment #8532842 - Flags: review?(smontagu)
(Assignee)

Updated

4 years ago
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
(Assignee)

Comment 2

4 years ago
Created attachment 8532844 [details] [diff] [review]
part 2 - Make Get{Min,Pref}ISize in video and plugin frames aware of vertical writing mode

Looking around in layout/generic, I notice the same issue exists for video and plugin frames; let's fix those while we're here.
Attachment #8532844 - Flags: review?(smontagu)
Attachment #8532842 - Flags: review?(smontagu) → review+
Attachment #8532844 - Flags: review?(smontagu) → review+
https://hg.mozilla.org/mozilla-central/rev/c3ba63a95999
https://hg.mozilla.org/mozilla-central/rev/e7d83c24264b
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.