Closed Bug 1907661 Opened 1 year ago Closed 1 year ago

[Toolbar Redesign] Button ripple effect is not applying theme correctly on some toolbar buttons

Categories

(Firefox for Android :: Toolbar, defect, P3)

Firefox 130
All
Android
defect

Tracking

()

RESOLVED FIXED
134 Branch
Tracking Status
firefox134 --- fixed

People

(Reporter: emanuellclaudiu, Assigned: tchoh)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxdroid][group3][toolbar-redesign-release-blocker])

Attachments

(2 files)

User Agent: Mozilla/5.0 (Android 11; Mobile; rv:130.0) Gecko/130.0 Firefox/130.0

Steps to reproduce:

Enable hidden setting: Navigation Toolbar. The buttons in the new Navigation toolbar have no color when pressed, on a dark theme. It is black color on tap, on dark theme. Only the tray with tabs has color when pressed, but when I press a little on the side it is also not noticeable.

Actual results:

The colors from pressing the buttons on the new Toolbar.

Expected results:

Buttons to display a white circle on dark theme.

Flags: needinfo?(cpeterson)

Thanks! Adding this bug to our Toolbar beta milestone.

Severity: -- → S4
Flags: needinfo?(cpeterson)
Priority: -- → P2
Summary: [Toolbar Redesign] The buttons in the new Navigation toolbar have no color when pressed, on a dark theme → [Toolbar Redesign] Button ripple effect is not applying theme correctly on some toolbar buttons
Assignee: nobody → tchoh
Status: NEW → ASSIGNED

Spoke with Petru. Might not be as easy of a fix as initial thoughts. Will time-box to 1 hr and see if there is a good fix. Otherwise, this ticket will be pushed back to a later time.

Whiteboard: [kitkat banana sprint]

Tim says this bug doesn't need to block our Beta milestone, but we should track it for our Release milestone (meta bug 1894470).

Roger says this issue might be fixed in the upcoming Compose 1.7 release, which is now in RC.

Priority: P2 → P3
Whiteboard: [kitkat banana sprint]
Whiteboard: [fxdroid][group3]
See Also: → 1910435
Whiteboard: [fxdroid][group3] → [fxdroid][group3][toolbar-redesign-release-blocker]
See Also: → 1931046
Pushed by zmckenney@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/281be4770155 Add Ripple color to FirefoxTheme and apply to navigation toolbar r=android-reviewers,petru
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: