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

MinimalFooter: allow buttons #264

Closed
stefankp opened this issue May 5, 2023 · 5 comments · Fixed by #272
Closed

MinimalFooter: allow buttons #264

stefankp opened this issue May 5, 2023 · 5 comments · Fixed by #272
Assignees
Labels

Comments

@stefankp
Copy link

stefankp commented May 5, 2023

We're looking to replace the custom Footer we have on some pages of https://resources.github.com/ with MinimalFooter. To do this, we need the option to allow button as alternative to MinimalFooter.Link.

Example: as you can see at the bottom of https://resources.github.com/enterprise/trial/, we have a button in the footer for "Manage cookies", which opens an overlay to manage cookie settings. The styling should be the same as the text link.

@rezrah
Copy link
Collaborator

rezrah commented May 9, 2023

Potential solution:

Enable as prop of type a | button for MinimalFooter.Link, similar to other polymorphic components in Primer Brand.

@stefankp
Copy link
Author

stefankp commented May 9, 2023

Works for me, but the button wouldn't have the href attribute.

@rezrah
Copy link
Collaborator

rezrah commented May 9, 2023

Works for me, but the button wouldn't have the href attribute.

👍 We'll handle that using conditional types

@JoshBowdenConcepts JoshBowdenConcepts self-assigned this May 10, 2023
@JoshBowdenConcepts
Copy link
Contributor

Just to clarify here! Is the intention for this content to be a Button but look like a Link, as you have shown on the Enterprise page?

@JoshBowdenConcepts JoshBowdenConcepts linked a pull request May 10, 2023 that will close this issue
9 tasks
@stefankp
Copy link
Author

stefankp commented May 11, 2023

@JoshBowdenConcepts exactly. Because it performs an action and doesn't lead to another URL.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants