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

fix: navset_card_pills/tabs/underline now properly uses header & footer args #1028

Merged
merged 6 commits into from
Aug 19, 2024

Conversation

tanho63
Copy link
Contributor

@tanho63 tanho63 commented Mar 27, 2024

Closes #1024

@CLAassistant
Copy link

CLAassistant commented Mar 27, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@gadenbuie gadenbuie left a comment

Choose a reason for hiding this comment

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

This looks great to me. We have some more work to do in this area, but this is very clearly an improvement over the current state of affairs.

(@cpsievert: @tanho63 and I talked about this at Tidyverse Dev Day.)

@@ -52,10 +52,7 @@ navset_card_pill <- function(

items <- collect_nav_items(..., wrapper = wrapper)
Copy link
Member

Choose a reason for hiding this comment

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

Note to self: we should probably be differentiating between children and attributes and sending attributes to card(). The other problem we might have here is that we should probably expose padding and gap options for the card we're creating but this is complicated by the fact that users can give padding and gap etc to nav_panel().

@gadenbuie
Copy link
Member

@tanho63 would you like to update this PR with main and move the NEWS bullet into the right place?

@tanho63
Copy link
Contributor Author

tanho63 commented Aug 15, 2024

@gadenbuie merged in latest main and fixed NEWS.md 😀

@tanho63 tanho63 requested a review from gadenbuie August 15, 2024 20:18
@cpsievert cpsievert merged commit 556e932 into rstudio:main Aug 19, 2024
10 of 11 checks passed
@cpsievert
Copy link
Collaborator

Thanks @tanho63!

@tanho63 tanho63 deleted the patch-1 branch August 19, 2024 21:10
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.

navset_card_pill, navset_card_tab, navset_card_underline ignore header arg
4 participants