Closed Bug 365715 Opened 18 years ago Closed 18 years ago

Text moves when selecting text with letter-spacing and UTF-8 characters

Categories

(Core :: Graphics, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: christoph.reiter, Assigned: masayuki)

References

Details

(Keywords: testcase)

Attachments

(2 files)

User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a2pre) Gecko/20070102 Minefield/3.0a2pre Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a2pre) Gecko/20070102 Minefield/3.0a2pre Text with letter-spacing and UTF-8 charakters in it moves when selecting. some things i experienced: if the letter-spacing is low (~1) it affects only the word with the UTF-8 character in it. many spaces reduce the effect Reproducible: Always Steps to Reproduce: 1. 2. 3.
Attached file testcase (deleted) —
i forgot something: sometimes if you open new tabs (not reproduceable for me) selecting works like it should. Then I have to reopen Firefox to reproduce it.
Component: General → GFX: Thebes
Keywords: testcase
Product: Firefox → Core
QA Contact: general → thebes
Version: unspecified → Trunk
can you attach the screenshot?
I can reproduce this bug, but I don't understand what happens.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch Patch rv1.0 (deleted) — Splinter Review
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
This is a regression of bug 357637. Now, we are using double cache that is gfxTextRunCache and local cache of gfxPangoTextRun (gfxPangoTextRun::mSegments). These caches are not cleared at changing the spacing. But this bug is not important and the patch conflicts to the patch of new textframe. Roc: Will you use the local cache in gfxPangoTextRun? If you don't cache it, the bug will be WFMed by your patch.
My patch should fix this, yeah. Spacing isn't cached in the textrun with my patch.
This bug is fixed by the roc's new textrun. -> FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Depends on: 333659
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: