Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: DCMAW 10331 Update copy on the Sign out confirmation page (with Wallet copy) #117

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

ThomasIent
Copy link
Contributor

@ThomasIent ThomasIent commented Nov 14, 2024

DCMAW-10331: Update copy on the Sign out confirmation page (with Wallet copy)

Description of changes:

  • Correct Typography migration in accordance with Android guidelines
    • Heading1 is DisplaySmall not HeadingLarge
    • Buttons use LabelLarge not LabelMedium
  • Separate and update m3 Colors
  • Move TERTIARY button type to QUATERNARY and use tertiary colors for new TERTIARY button
  • Add Error button type
  • Match Figma designs for the GDSButtons (no shadows yet though)
  • Simplify the ButtonParameters

Evidence of the change

Old Buttons

Screenshot 2024-11-15 at 12 06 30
Screenshot 2024-11-15 at 12 06 39
Screenshot 2024-11-15 at 12 06 53
Screenshot 2024-11-15 at 12 07 02
Screenshot 2024-11-15 at 12 07 10
Screenshot 2024-11-15 at 12 07 17

New Buttons

DCMAW-10331-Buttons-Dark
DCMAW-10331-Buttons-Light

Old Colors

DCMAW-10331-Old-Colors

New Colors

DCMAW-10331-New-Colors

Old Headings

Screenshot 2024-11-15 at 12 10 10
Screenshot 2024-11-15 at 12 10 18

New Headings

Screenshot 2024-11-15 at 12 14 00
Screenshot 2024-11-15 at 12 14 12

Checklist

Before creating the pull request

  • Commit messages that conform to conventional commit messages.
  • Ran the app locally ensuring it builds.
  • Tests pass locally.
  • Pull request has a clear title with a short description about the feature or update.
  • Created a draft pull request if it's not ready for review.

Before the CODEOWNERS review the pull request

  • Complete all Acceptance Criteria within Jira ticket.
  • Self-review code.
  • Successfully run changes on a testing device.
  • Complete automated Testing:
    • Unit Tests.
    • Integration Tests.
    • Instrumentation / Emulator Tests.
  • Review Accessibility considerations.
  • Handle PR comments.

Before merging the pull request

Update m3 Typography to align with what is set for m2 based on the migration documentation
Update m3 Typography to include a preview of all text styles

Resolves: 10331
Rename Shape to Shapes to match m2
Update m3 Shapes to align with what is set for m2 based on the migration documentation
Update m3 Shapes to include a preview of all shapes

Resolves: 10331
Add ColorScheme to pull colors out of Theme
Add small documentation on how m2 and m3 colors compare

Resolves: 10331
Remove the use of the Keep annotation and add proguard rules instead

Resolves: 10331
Add CloseButton composable to the components m3 buttons package
Add CloseButtonTest to test CloseButton
Update strings.xml to include content description
Update Dimensions to include closeButtonSize resource

Resolves: 10331
Update Buttons to more closely match the Figma designs
Update ButtonType to include ERROR type, swap TERTIARY to QUATERNARY (it doesn't use tertiary colors)
Update ButtonParameters to remove unused parameters
Update ButtonProvider to group ButtonParameters variations by ButtonType

This has required updating Color, ColorScheme, Typography, and Dimensions

Resolves: 10331
Delete resources for non-compose ui

Resolves: 10331
Update build.gradle.kts to include android orchestrator

Resolves: 10331
Add ShapesTest to cover the Shapes compose preview
Update Shapes with a small bit of tidying
Update Theme to fix a small mistake

Resolves: 10331
Update BulletList to remove usage of M3 classes

Resolves: 10331
Rename M3 HeadingSize Classes to match M3 naming convention

Resolves: 10331
Delete duplicate drawable resources in pages
Auto-layout all drawable resources
(Small format change in build.gradle.kts)

Resolves: 10331
Resolves: 10331
@ThomasIent ThomasIent marked this pull request as ready for review November 15, 2024 17:49
@ThomasIent ThomasIent requested review from a team as code owners November 15, 2024 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant