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

Link to patch detail #2741

Merged
merged 5 commits into from
Jul 11, 2024
Merged

Link to patch detail #2741

merged 5 commits into from
Jul 11, 2024

Conversation

dottorblaster
Copy link
Contributor

@dottorblaster dottorblaster commented Jul 3, 2024

This PR adds:

  • a new button type called link used to make standard effortless programmatic navigation without having to appease a11y guidelines every time
  • navigation from SUSE Manager overviews to advisory detail view

@dottorblaster dottorblaster self-assigned this Jul 3, 2024
@dottorblaster dottorblaster force-pushed the link-patch-detail branch 2 times, most recently from 63373db to a3c3e96 Compare July 9, 2024 07:27
@dottorblaster dottorblaster marked this pull request as ready for review July 11, 2024 09:13
@balanza
Copy link
Member

balanza commented Jul 11, 2024

Please provide PR description

@dottorblaster
Copy link
Contributor Author

@balanza I totally overlooked that, thank you

(buttonSize) => {
const content = faker.vehicle.vehicle();
render(
<Button size={buttonSize} disabled>
Copy link
Contributor

Choose a reason for hiding this comment

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

Should the button be disabled here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

LOL nope my bad, thanks

Copy link
Member

@janvhs janvhs left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks for improving the button story as well!

@dottorblaster dottorblaster merged commit e60825f into main Jul 11, 2024
26 checks passed
@dottorblaster dottorblaster deleted the link-patch-detail branch July 11, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants