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

WSTEAM1-1011 UGC Form Submit Button #11613

Merged
merged 19 commits into from May 14, 2024
Merged

Conversation

emilysaffron
Copy link
Contributor

@emilysaffron emilysaffron commented May 9, 2024

Resolves JIRA WSTEAM1-1011

Overall changes

Adds styling for submit button

Code changes

  • Adds Service neutral core to palette
  • Styles submit button

Testing

  1. visit http://localhost.bbc.com:7081/mundo/send/u50853489?renderer_env=live and check it matches the designs for stage 1 , button designs
  2. Check it matches a11y and screenreader UX

Helpful Links

Add Links to useful resources related to this PR if applicable.

Coding Standards

Repository use guidelines

@emilysaffron emilysaffron self-assigned this May 9, 2024
@emilysaffron emilysaffron marked this pull request as ready for review May 13, 2024 08:12
@karinathomasbbc
Copy link
Collaborator

Is the focus indicator correct? This is the Submit button with tab order. I was expecting a black line around the entire button per the interactions tab on Figma.

image

@karinathomasbbc
Copy link
Collaborator

Also (perhaps a question for @amoore108 or @andrewscfc), do we need to do anything about disabling the Submit button functionality when viewing the UGC Page on Storybook?

@emilysaffron
Copy link
Contributor Author

Is the focus indicator correct? This is the Submit button with tab order. I was expecting a black line around the entire button per the interactions tab on Figma.

image

Oh weird, what browser are you checking on ? when i focus on it with tab it looks like this for me
Screenshot 2024-05-13 at 09 31 27

@amoore108
Copy link
Contributor

Also (perhaps a question for @amoore108 or @andrewscfc), do we need to do anything about disabling the Submit button functionality when viewing the UGC Page on Storybook?

Not really sure tbh. The UGC API is only accessible on authorised domains, so Storybook will always fail. Don't think its a huge problem as it won't do anything if someone tries to make a POST request from it.

@amoore108
Copy link
Contributor

Testing Firefox I get the same focus indicator issue @karinathomasbbc had. Chrome seems fine though 🤔

@karinathomasbbc
Copy link
Collaborator

Testing Firefox I get the same focus indicator issue @karinathomasbbc had. Chrome seems fine though 🤔

Oops forgot to mention - yes, this was on Firefox, using the Storybook Permalink.

@emilysaffron
Copy link
Contributor Author

Testing Firefox I get the same focus indicator issue @karinathomasbbc had. Chrome seems fine though 🤔

Oops forgot to mention - yes, this was on Firefox, using the Storybook Permalink.

okay this should be fixed now, not entirely sure why the global styling for the focus indicator wasnt working for firefox, but if i just paste it explicitly into the css file it works 🤷 weird

@Isabella-Mitchell
Copy link
Contributor

Not sure if relevant but I remember Firefox being a bit odd sometimes with the focus indicator.
It's possible this fix helps? (giving accessibility.tabfocus a value of 7) - https://support.mozilla.org/en-US/questions/1278793

@emilysaffron
Copy link
Contributor Author

Testing Firefox I get the same focus indicator issue @karinathomasbbc had. Chrome seems fine though 🤔

Oops forgot to mention - yes, this was on Firefox, using the Storybook Permalink.

okay this should be fixed now, not entirely sure why the global styling for the focus indicator wasnt working for firefox, but if i just paste it explicitly into the css file it works 🤷 weird

figured it out, firefox has default styling for elements of type 'submit' that needed to be overwritten

Copy link
Contributor

@amoore108 amoore108 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job!

Copy link
Contributor

@Isabella-Mitchell Isabella-Mitchell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fab - thanks for making those changes :)

@emilysaffron emilysaffron merged commit cc4dbcd into latest May 14, 2024
11 checks passed
@emilysaffron emilysaffron deleted the WSTEAM1-1011-UGC-submit-button branch May 14, 2024 08:45
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.

None yet

4 participants