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

use sphinx-design tabs #2363

Merged
merged 11 commits into from
Nov 3, 2023
Merged

Conversation

GrahamSH-LLK
Copy link
Contributor

@GrahamSH-LLK GrahamSH-LLK commented Oct 24, 2023

incomplete, I still have 15 files to go
however the first portion is done and I'm heading to bed

source/docs/contributing/frc-docs/style-guide.rst Outdated Show resolved Hide resolved
@GrahamSH-LLK
Copy link
Contributor Author

Should be done/ ready for review

@sciencewhiz
Copy link
Collaborator

It would be good to add something to the style guide about how to use tabs. It's not there now, but since the methods are changing I think it makes sense to document now

@Daltz333
Copy link
Member

Do the tab-set-code actually sync across all tabs across a page?

@sciencewhiz
Copy link
Collaborator

I had tested a few and they did, but I haven't tried multiple browsers

@sciencewhiz sciencewhiz changed the title use new tabs use sphinx-design tabs Oct 28, 2023
@sciencewhiz
Copy link
Collaborator

sphinx-tabs can be removed as a dependency for poetry, right?

@GrahamSH-LLK
Copy link
Contributor Author

sphinx-tabs can be removed as a dependency for poetry, right?

yes

@GrahamSH-LLK
Copy link
Contributor Author

I added to style guide and fixed the conflicts

@Daltz333 Daltz333 merged commit 5a9c005 into wpilibsuite:main Nov 3, 2023
9 checks passed
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.

3 participants