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

Warning button disabled styling #1034

Open
rowellx68 opened this issue Sep 27, 2024 · 0 comments · May be fixed by #1051
Open

Warning button disabled styling #1034

rowellx68 opened this issue Sep 27, 2024 · 0 comments · May be fixed by #1051
Labels
🐛 bug Something isn't working the way it should (including incorrect wording in documentation)

Comments

@rowellx68
Copy link

Bug Report

What is the issue?

When adding both the nhsuk-button--disabled class and the disabled attribute to a warning button, the button's colour changes from red to green. This issue also occurs when we just use the disabled attribute. We expect this to behave like how it does when only the nhsuk-button--disabled is used.

What steps are required to reproduce the issue?

  1. Create a warning button
  2. Add the nhsuk-button--disabled class and disabled attribute

What was the environment where this issue occurred?

  • Device: N/A
  • Operating System: N/A
  • Browser: N/A
  • Browser version: N/A
  • NHS.UK frontend package version: 8.3 and 9
  • Node version: N/A
  • npm version: N/A

Is there anything else you think would be useful in recreating the issue?

nhsuk-button-styles

@anandamaryon1 anandamaryon1 added the 🐛 bug Something isn't working the way it should (including incorrect wording in documentation) label Sep 30, 2024
@paulrobertlloyd paulrobertlloyd linked a pull request Oct 22, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working the way it should (including incorrect wording in documentation)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants