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

Explain why start buttons are output as HTML links, not buttons #1372

Merged
merged 5 commits into from Oct 27, 2020

Conversation

timpaul
Copy link
Contributor

@timpaul timpaul commented Oct 7, 2020

In response to this query in the backlog: alphagov/govuk-design-system-backlog#34 (comment)

@timpaul timpaul added awaiting triage Needs triaging by team button labels Oct 7, 2020
@govuk-design-system-ci
Copy link
Collaborator

govuk-design-system-ci commented Oct 7, 2020

You can preview this change here:

Built with commit 46a1cc0

https://deploy-preview-1372--govuk-design-system-preview.netlify.app

@timpaul
Copy link
Contributor Author

timpaul commented Oct 7, 2020

Hmm, as per Ed's comment here and his issue here, we could go ever further and provide an explicit example of a link styled as a button.

Based on feedback from Joe L in PR
@timpaul timpaul removed the awaiting triage Needs triaging by team label Oct 12, 2020
@timpaul timpaul added this to Needs review in Design System Sprint Board via automation Oct 12, 2020
@trang-erskine trang-erskine marked this pull request as ready for review October 21, 2020 13:22
@timpaul timpaul requested review from trang-erskine and joelanman and removed request for StephenGill and joelanman October 21, 2020 13:27
Copy link
Contributor

@trang-erskine trang-erskine left a comment

Choose a reason for hiding this comment

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

Nikki suggested splitting it into 2 sentences. The 2nd option removes "are rendered" which is my preferred option.

"Start buttons are rendered as links rather than inputs. This is because your users are not submitting any data."

"Start buttons are links rather than inputs. This is because your user are not submitting any data."

@timpaul
Copy link
Contributor Author

timpaul commented Oct 21, 2020

I realised after the review that the wording is incorrect - default buttons use the button HTML element, not the input one.

In the end I chose to be explicit and name the actual HTML tags involved, to avoid the potential confusion of saying "Start buttons are links not buttons".

@timpaul timpaul changed the title Explain why start buttons are rendered as links Explain why start buttons are output as HTML links, not buttons Oct 21, 2020
Copy link
Contributor

@trang-erskine trang-erskine left a comment

Choose a reason for hiding this comment

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

Additional information about the start button looks good.

To prevent them being interpreted as invalid HTML code
Reduce repetition and use a single sentence instead of two.
@timpaul timpaul merged commit a4141f7 into master Oct 27, 2020
Design System Sprint Board automation moved this from Needs review to Done Oct 27, 2020
@timpaul timpaul deleted the timpaul-patch-6 branch October 27, 2020 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants