cds-snc / platform-forms-client

NextJS application that serves the public-facing website for Forms
https://forms-staging.cdssandbox.xyz/
MIT License
34 stars 13 forks source link

GCDS: Button #4307

Open samsadasivan opened 4 days ago

samsadasivan commented 4 days ago

Audit:

DESKTOP

*GCDS coded button is not aligned with our mobile button sizing in figma and fonts + padding do not scale down like the Forms buttons

MOBILE

Current state

image.png

Desired state

image.png
timarney commented 3 days ago

PR:

Forms

https://github.com/user-attachments/assets/0a5817e8-4da8-4e60-8ea1-f181d214a6b2

GCDS

https://github.com/user-attachments/assets/9a34b695-8b38-4456-9b87-b299b029ade4

https://github.com/cds-snc/platform-forms-client/pull/4319

timarney commented 3 days ago

Noting the same button component is used on non front facing forms --- so possibly might need to make adjustments as we find them inside the builder.

i.e. add element ---- in this case the active state matches the black bkd now https://github.com/user-attachments/assets/8da861f0-1a76-4b1a-901e-b95c0632d81e

timarney commented 3 days ago

so possibly might need to make adjustments as we find them inside the builder.

For example the Accounts "filters"

Screenshot 2024-09-24 at 11 16 28 AM
timarney commented 3 days ago

https://github.com/user-attachments/assets/a9edcfa6-2a67-4cfb-a386-fd521de73000