Closed Bug 1784282 Opened 2 years ago Closed 2 years ago

[Colorway Closet] The colorway is not applied at some point after cycling through the colorway radios by holding down arrow keys for multiple seconds

Categories

(Firefox :: Theme, defect, P3)

Firefox 105
Desktop
All
defect
Points:
3

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox103 --- disabled
firefox104 --- disabled
firefox105 --- affected

People

(Reporter: ailea, Unassigned)

References

Details

(Whiteboard: [fidefe-colorway-closet])

Attachments

(1 file, 1 obsolete file)

Attached video 2022-08-11_16h33_49.mp4 (deleted) —

Found in

Firefox 105.0a1

Affected versions

104.0a1
105.0a1

Affected platforms

Windows 10
Ubuntu 20

Preconditions

In about:config set:
"browser.theme.colorway-closet" to true
"browser.tabs.firefox-view" to true

Steps to reproduce

  1. Open Colorways modal from Fxview or from about:addons
  2. Using the keyboard, get any random colorway in focus, keep left or right keyboard arrow for 2-3 seconds and release it.

Expected result

The colorway in focus is not applied.

Actual result

The colorway selected should be applied accordingly.

Regression range

Not a regression.

Note:

The issue seems to be intermittent, I can reproduce it around 2-3 attempts from 5. It is not reproducible at all on macOS.

Has STR: --- → yes
Points: --- → 2
Priority: -- → P3
Summary: [Colorway Closet] The colorway is not applied after navigating through all of them using the keyboard → [Colorway Closet] The colorway is not applied at some point after cycling through the colorway radios by holding down arrow keys for multiple seconds
Whiteboard: [fidefe-colorway-closet]
Points: 2 → 3

Some observations while reproducing this bug on Windows 10: the failure occurs when we try to read an extension ID because extension is null. We pass data about the colorway here, which is especially obtained via getData (call, definition). getData is returning null however, causing the error and failing to load a colorway.

Assignee: nobody → kpatenio
Status: NEW → ASSIGNED

When a theme is enabled and then disabled (or vice-versa) really quickly, there is a chance that the theme will fail to load due to a race condition. To address this issue on the colorway closet modal, delay keyboard arrow input on colorway family or intensity radio buttons - this will give the browser time to fully load a selected colorway before trying to view and enable another one.

Attachment #9300346 - Attachment description: WIP: Bug 1784282 - delay colorway radio button navigation until colorway is applied for colorway closet. → Bug 1784282 - delay colorway radio button navigation until colorway is applied for colorway closet.
Assignee: kpatenio → nobody
Status: ASSIGNED → NEW
Attachment #9300346 - Attachment is obsolete: true

Can this be resolved wontfix/invalid Amy? I don't think the interaction described will be possible anymore, right?

Flags: needinfo?(achurchwell)
Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(achurchwell)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: