Closed
Bug 1879035
Opened 7 months ago
Closed 1 month ago
Unable to Free highlight over an already Highlighted Text
Categories
(Firefox :: PDF Viewer, defect, P1)
Tracking
()
RESOLVED
FIXED
130 Branch
People
(Reporter: rdoghi, Assigned: calixte)
References
Details
Attachments
(3 files)
Found in
- Nightly 124.0a1 (2024-02-07)
Affected versions
- Nightly 124.0a1 (2024-02-07)
Affected platforms
- All
Steps to reproduce
- Open any PDF in Firefox.
- Highlight any text.
- Start to Free Highlight from a non text area until the Highlighted text.
Expected result
- Users should be able to free Highlight on top of other Highlighted text.
Actual result
- The Free highlight stops working if it crosses an already Highlighted text.
Regression range
Not a Regression
Comment 1•7 months ago
|
||
:rdoghi, if you think that's a regression, could you try to find a regression range using for example mozregression?
Assignee | ||
Comment 2•1 month ago
|
||
Assignee | ||
Updated•1 month ago
|
Assignee: nobody → cdenizet
Status: NEW → ASSIGNED
Priority: -- → P1
Reporter | ||
Comment 3•1 month ago
|
||
This issue is Verified as fixed in our latest Nightly build 130.0a1 (2024-07-17) by Bug 1907917.
status-firefox128:
--- → affected
status-firefox129:
--- → affected
status-firefox130:
--- → verified
Updated•1 month ago
|
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
No longer depends on: 1907705
Resolution: --- → FIXED
Target Milestone: --- → 130 Branch
Comment 4•1 month ago
|
||
The patch landed in nightly and beta is affected.
:calixte, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox129
towontfix
.
For more information, please visit BugBot documentation.
Flags: needinfo?(cdenizet)
Assignee | ||
Comment 5•29 days ago
|
||
Updated•29 days ago
|
Attachment #9414124 -
Flags: approval-mozilla-beta?
Comment 6•29 days ago
|
||
beta Uplift Approval Request
- User impact if declined: Small
- Code covered by automated testing: yes
- Fix verified in Nightly: no
- Needs manual QE test: yes
- Steps to reproduce for manual QE testing: See comment#0
- Risk associated with taking this patch: Small
- Explanation of risk level: It can only happen if a user tries to highlight some text and the pointer hovers an existing editor. So it should be pretty rare.
- String changes made/needed: No
- Is Android affected?: no
Flags: qe-verify+
Assignee | ||
Updated•29 days ago
|
Flags: needinfo?(cdenizet)
Updated•28 days ago
|
Updated•28 days ago
|
Updated•28 days ago
|
Attachment #9414124 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•28 days ago
|
Updated•28 days ago
|
QA Whiteboard: [qa-triaged]
Reporter | ||
Comment 8•28 days ago
|
||
Verified as fixed in in 129.0b8.
You need to log in
before you can comment on or make changes to this bug.
Description
•