Closed Bug 1154227 Opened 10 years ago Closed 10 years ago

[CSS3] With writing-mode: vertical-rl, text-shadow is only applied to part of the text block

Categories

(Core :: Graphics, defect)

39 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: kristin.merrigan, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

(Keywords: css3, testcase, Whiteboard: [gfx-noted])

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.118 Safari/537.36 Steps to reproduce: used following html to apply a text shadow with blur effect for vertical text (using the developer version 39.0a2 (2015-04-13) which supports css styles writing-mode: vertical-rl style) <!DOCTYPE html> <html> <head> <style> h1 { text-shadow: 0px 0px 2px #FF0000; writing-mode: vertical-rl; text-orientation: upright; } </style> </head> <body> <h1>Text-shadow effect</h1> </body> </html> Actual results: it blurs slightly around the first character then gets cut off, and no other characters have the effect applied. Expected results: It should apply the blur effect to all characters
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:40.0) Gecko/20100101 Firefox/40.0 Build ID: 20150413030203 I could reproduce the issue without text-orientation: upright. HWA on or off made no difference. Neither did opening the tab in a non-e10s window.
Status: UNCONFIRMED → NEW
Component: Untriaged → Graphics
Ever confirmed: true
Keywords: css3, testcase
Product: Firefox → Core
Summary: css style text-shadow with blur effect does not work properly when used on vertical text (using the developer version) → [CSS3] With writing-mode: vertical-rl, text-shadow is only applied to part of the text block
Version: unspecified → 39 Branch
Part of the problem, at least, is that we don't convert the bounding box from textrun-relative to physical coordinates in nsTextFrame::PaintShadows.
Blocks: writing-mode
Whiteboard: [gfx-noted]
Here's a more extensive testcase with combinations of writing-mode and text-orientation.
And with this patch, all the shadows in the testcase render properly in vertical mode.
Attachment #8595309 - Flags: review?(smontagu)
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Attachment #8595309 - Flags: review?(smontagu) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Interactive DHTML text-shadow and writing-mode test, testing 'text-shadow' property with all combinations of 'text-orientation' values and of 'writing-mode' values http://www.gtalbot.org/BrowserBugsSection/CSS3WritingModes/s76-text-shadow-dhtml.xht
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: