Closed
Bug 1987091
Opened 2 months ago
Closed 1 month ago
Migrate some components to use the M3 button color defaults
Categories
(Firefox for Android :: Logins, task, P2)
Tracking
()
VERIFIED
FIXED
146 Branch
People
(Reporter: mlobontiuroman, Assigned: giorga)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid][group4])
Attachments
(1 file)
|
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
Steps to reproduce:
- Go to Settings - Autofill - Addresses, and add an address.
- Tap on the saved address, and tap on the delete icon.
- Go to Settings - Passwords, and save a password.
- Go to the saved password, and tap on the three-dot menu -> "Delete".
- Go to Settings - Autofill - Manage cards, and save at least one card.
- Tap on a saved card, and select the delete icon.
Actual results
The following dialogues displayed after steps 2, 4, and 6 don't match the M3 design:
- the "Delete this address? CANCEL DELETE",
- "Are you sure you want to delete this password? CANCEL DELETE",
- the "Delete card? CANCEL DELETE".
Expected results
The usages of TextButtons have been updated to follow the colours in the new specs found here. There should be no uppercase spec.
Additional details
- Nightly 144.0a1 from 9/4
Updated•2 months ago
|
Updated•2 months ago
|
Severity: -- → N/A
Priority: -- → P2
| Assignee | ||
Updated•1 month ago
|
Assignee: nobody → giorga
| Assignee | ||
Comment 1•1 month ago
|
||
Updated•1 month ago
|
Status: NEW → ASSIGNED
Pushed by giorga@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/67cd6f3e3d6c
https://hg.mozilla.org/integration/autoland/rev/a73eba47100f
The dialog's buttons should be in lowercase. r=android-reviewers,007
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
status-firefox146:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 146 Branch
Updated•1 month ago
|
Comment 4•1 month ago
|
||
Should we uplift this to Beta? Please nominate if so.
Flags: needinfo?(giorga)
| Assignee | ||
Comment 5•28 days ago
|
||
Comment on attachment 9518675 [details]
Bug 1987091 - The dialog's buttons should be in lowercase.
Beta/Release Uplift Approval Request
- User impact if declined/Reason for urgency: The dialog's buttons will be in uppercase.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky):
- String changes made/needed:
- Is Android affected?: Yes
Flags: needinfo?(giorga)
Attachment #9518675 -
Flags: approval-mozilla-beta?
| Assignee | ||
Updated•28 days ago
|
Flags: qe-verify+
Updated•28 days ago
|
Attachment #9518675 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•28 days ago
|
Comment 7•27 days ago
|
||
Verified as fixed on Fenix 145.0b4 using Lenovo Yoga Tab 11 (Android 12) and Google Pixel 7 Pro (Android 14).
The Cancel and Delete buttons now follow the Material 3 default button colors.
Updated•27 days ago
|
Updated•27 days ago
|
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•