Closed
Bug 1869775
Opened 1 year ago
Closed 1 year ago
The bookmark tag checkbox is not visible
Categories
(Firefox :: Theme, defect)
Tracking
()
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
firefox-esr115 | --- | unaffected |
firefox120 | --- | unaffected |
firefox121 | --- | unaffected |
firefox122 | --- | affected |
People
(Reporter: atrif, Unassigned)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
Found in
- 122.0a1 (2023-12-13)
Affected versions
- 122.0a1 (2023-12-13)
Tested platforms
- Affected platforms: Windows 10, macOS 12, Ubuntu 22
- Unaffected platforms: none
Steps to reproduce
- Open Firefox and a random page and save it as a bookmark.
- Add a tag for the previously saved bookmark.
- Open that page and click on the
Edit this bookmark button
. - Click on the tag (if multiple tags are present click on the empty tag area)
Expected result
- The checkbox is visible.
Actual result
- Only the checkmark is visible.
Regression
- Last good revision: 2462e8026fd78d968362eacf10e6ae2dee025e1e
First bad revision: d5a2cb7b9272b5c9cdfe4a0397c570ce5f9260bf
Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=2462e8026fd78d968362eacf10e6ae2dee025e1e&tochange=d5a2cb7b9272b5c9cdfe4a0397c570ce5f9260bf
Additional notes
- Attached a screen recording.
- The regression applies only to Edit this bookmark menu from the URL bar.
Comment 1•1 year ago
|
||
:emilio, since you are the author of the regressor, bug 1867769, could you take a look?
For more information, please visit BugBot documentation.
Flags: needinfo?(emilio)
Comment 2•1 year ago
|
||
This looks fairly ok, specially given that menu is not used all that much. Atrif, what was the behavior before my patch?
Flags: needinfo?(emilio) → needinfo?(atrif)
Reporter | ||
Comment 3•1 year ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #2)
This looks fairly ok, specially given that menu is not used all that much. Atrif, what was the behavior before my patch?
Hello Emilio! The checkbox had a darker blue color. See the attached screen recording.
Flags: needinfo?(atrif)
Comment 4•1 year ago
|
||
This is fine as-is given that each of these rows acts as a click target for the embedded checkbox.
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•