Closed Bug 1900127 Opened 6 months ago Closed 6 months ago

UX Fit-And-Finish Style Recommendations

Categories

(Firefox :: Translations, task, P3)

task

Tracking

()

RESOLVED FIXED
128 Branch
Tracking Status
firefox128 --- fixed

People

(Reporter: nordzilla, Assigned: nordzilla)

References

Details

Attachments

(3 files)

Description

Dasha has given me some visual Fit-and-Finish changes to make for the panel before release.


Steps to implement

  • Modify and clean up the CSS to better match the exact specifications given above.

Tests to implement

  • These are visual changes only. Preexisting tests are sufficient coverage.

Removes the default focus outline when encountering error
states in the SelectTranslationsPanel, such as on the
try-again button. UX recommendation is that the button
is focused, but not outlined, unless the user tabbed to it.

Cleans up the state of the SelectTranslationsPanel markup
as part of the Fit-and-Finish UX review before release.

Depends on D212739

Cleans up the SelectTranslationsPanel CSS to both
use spacing variables as well as more closely match
the margin spacing that is specified in the Figma.

Depends on D212740

Attachment #9405772 - Attachment description: WIP: Bug 1900127 - Remove Select Translations default focus outlines r=#translations-reviewers! → WIP: Bug 1900127 - Clean up SelectTranslations default states r=#translations-reviewers!
Attachment #9405772 - Attachment description: WIP: Bug 1900127 - Clean up SelectTranslations default states r=#translations-reviewers! → Bug 1900127 - Clean up SelectTranslations default states r=#translations-reviewers!
Attachment #9405773 - Attachment description: WIP: Bug 1900127 - Clean up SelectTranslationsPanel markup r=#translations-reviewers! → Bug 1900127 - Clean up SelectTranslationsPanel markup r=#translations-reviewers!
Attachment #9405774 - Attachment description: WIP: Bug 1900127 - Fix SelectTranslationsPanel margins r=#desktop-theme-reviewers! → Bug 1900127 - Fix SelectTranslationsPanel margins r=#desktop-theme-reviewers!
Attachment #9405772 - Attachment description: Bug 1900127 - Clean up SelectTranslations default states r=#translations-reviewers! → Bug 1900127 - Clean up SelectTranslations default states r=#translations-reviewers!,ohall
Attachment #9405773 - Attachment description: Bug 1900127 - Clean up SelectTranslationsPanel markup r=#translations-reviewers! → Bug 1900127 - Clean up SelectTranslationsPanel markup r=#translations-reviewers!,ohall
Attachment #9405774 - Attachment description: Bug 1900127 - Fix SelectTranslationsPanel margins r=#desktop-theme-reviewers! → Bug 1900127 - Clean up SelectTranslationsPanel margins r=#desktop-theme-reviewers!
Attachment #9405774 - Attachment description: Bug 1900127 - Clean up SelectTranslationsPanel margins r=#desktop-theme-reviewers! → Bug 1900127 - Fix SelectTranslationsPanel margins r=#desktop-theme-reviewers!
Attachment #9405774 - Attachment description: Bug 1900127 - Fix SelectTranslationsPanel margins r=#desktop-theme-reviewers! → Bug 1900127 - Clean up SelectTranslationsPanel margins r=#desktop-theme-reviewers!
Pushed by enordin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9828d6d4e8f6 Clean up SelectTranslations default states r=translations-reviewers,ohall https://hg.mozilla.org/integration/autoland/rev/5de2382a319c Clean up SelectTranslationsPanel markup r=translations-reviewers,ohall https://hg.mozilla.org/integration/autoland/rev/b611dca1d8ac Clean up SelectTranslationsPanel margins r=desktop-theme-reviewers,dao
Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
Regressions: 1902254
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: