Size of new Gecko selection carets doesn't take font inflation into account

RESOLVED WORKSFORME

Status

()

Firefox for Android
Text Selection
RESOLVED WORKSFORME
2 years ago
2 years ago

People

(Reporter: JanH, Unassigned)

Tracking

41 Branch
All
Android
Points:
---

Firefox Tracking Flags

(fennecNightly+)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Created attachment 8611233 [details]
example of selection carets being too small after font inflation

The size of the new Gecko selection carets (from bug 988143) seems to scale with the current zoom level. In conjunction with font inflation, this leads to an unwanted side effect: When font inflation kicks in and bumps up the font size, the text becomes readable, but the selection carets remain at their old size and are consequently somewhat too small - see attached example.
(Reporter)

Comment 1

2 years ago
Just noticed: This behaviour only happens if just "touchcaret.enabled" and "selectioncaret.enabled" are set to true, as per https://mail.mozilla.org/pipermail/mobile-firefox-dev/2015-May/001197.html.

If I also enable "layout.accessiblecaret.enabled", the selection carets are displayed at a sensible size. If the ultimate plan is to have all three options enabled by the default, I guess this bug isn't needed after all.
The caret size is retained only on AccessibleCaret, not on TouchCaret or SelectionCarets. 

AccessibleCaret provides the same functionality as the combination of TouchCaret and SelectionCarets. To avoid any event handling conflict, TouchCaret and SelectionCarets will be disabled when AccessibleCarets are enabled [1]. The ultimate goal is to enable AccessibleCaret and to disable TouchCaret and SelectionCarets.

[1] https://dxr.mozilla.org/mozilla-central/source/layout/base/nsPresShell.cpp?from=nsPresShell.cpp#899
Blocks: 988143
Status: UNCONFIRMED → NEW
tracking-fennec: --- → ?
Ever confirmed: true
tracking-fennec: ? → Nightly+
Depends on: 776030
Not sure why this was made dependent on fennec-apz, so removing dependency. Feel free to re-add it with an explanation of the connection.
No longer depends on: 776030
We need to discuss this again.
tracking-fennec: Nightly+ → ?

Comment 5

2 years ago
I don't see a bug filed about the handles zooming with the page zoom, but it seems to me that this is just a symptom of that core problem. Capella, is there a bug filed for that somewhere?

I think that the handles should always be the same size, regardless of zoom level or font inflation, and that size should be an appropriately-sized tappable area.
Flags: needinfo?(markcapella)

Comment 6

2 years ago
(In reply to Mark Finkle (:mfinkle) from comment #4)
> We need to discuss this again.

What did you want to discuss? This is an issue held on Nightly, so we decided to reset to Nightly+
tracking-fennec: ? → Nightly+
Flags: needinfo?(mark.finkle)
I don't believe there's a bug/request to modify Gecko caret sizing behaviour for the Android, nor for the base FF/OS, implementation.
Flags: needinfo?(markcapella)
Bug 1068525 is the one for keeping the caret handles always be the same size for the new AccessibleCaret, and I've enabled the the AccessibleCaret on B2G by default in bug 1172382. So B2G doesn't have this issue.
(In reply to :Margaret Leibovic from comment #6)
> (In reply to Mark Finkle (:mfinkle) from comment #4)
> > We need to discuss this again.
> 
> What did you want to discuss? This is an issue held on Nightly, so we
> decided to reset to Nightly+

This was made Nightly+ becase we assumed it was blocked by the Gecko APZ work, but it is not. And now we need to figure out the relationship of AccessibleCaret, TouchCaret and SelectionCarets.
Flags: needinfo?(mark.finkle)
(Reporter)

Comment 10

2 years ago
The new implementation on Nightly is displaying the carets at the same size regardless of zoom level or font inflation, so I guess this is no longer relevant.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.