Closed
Bug 1600652
Opened 5 years ago
Closed 5 years ago
Floating text selection toolbar misplaced above the selection
Categories
(Core :: Graphics: WebRender, defect, P1)
Tracking
()
VERIFIED
FIXED
mozilla73
| Tracking | Status | |
|---|---|---|
| firefox73 | --- | verified |
People
(Reporter: ekager, Assigned: ktaeleman)
References
Details
(Whiteboard: [geckoview:m1912])
Attachments
(2 files)
toolbar.png
STR:
- Select text in Fenix/RB Nightly
- Floating toolbar appears way above the selected text
Looks like a recent regression in Nightly
Fenix issue: https://github.com/mozilla-mobile/fenix/issues/6877
I was able to reproduce this on github.
Priority: -- → P1
Whiteboard: [geckoview:m1912]
|
||
Comment 2•5 years ago
|
||
This only seems to happen with WebRender enabled. Can you take a look :kris?
Flags: needinfo?(ktaeleman)
|
Assignee | |
Updated•5 years ago
|
Blocks: wr-android-mvp
Flags: needinfo?(ktaeleman)
|
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → ktaeleman
|
|
||
Updated•5 years ago
|
Component: General → Graphics: WebRender
Product: GeckoView → Core
|
|
Assignee | |
Comment 3•5 years ago
|
||
Look like the same bug was present in non-wr codepath:
https://bugzilla.mozilla.org/show_bug.cgi?id=1509271
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
||
Updated•5 years ago
|
Attachment #9117603 -
Attachment is obsolete: true
|
|
||
Updated•5 years ago
|
Attachment #9117603 -
Attachment is obsolete: false
Pushed by ktaeleman@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d89901d9bd4
Update Scroll frame metrics for android so text select popup is in correct location. r=botond
|
||
Comment 6•5 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox73:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
|
||
Comment 7•5 years ago
|
||
Hello, I can confirm that this issue is no longer reproducible using the latest version of Firefox Preview Nightly 1/6 #200106.
Devices:
- Nexus 5 (Android 6.0.1);
- Nokia 6 (Android 7.1.1);
- Google Pixel (Android 9).
Due to my findings, I'll set the status to Verified, thanks.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•