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.

Sign up for GitHub

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

Jump to bottom

Update topic and contents directive style #785

Merged
merged 3 commits into from
Jul 7, 2022
Merged

Update topic and contents directive style #785

merged 3 commits into from
Jul 7, 2022

Conversation

choldgraf
Copy link
Collaborator

@choldgraf choldgraf commented Jul 5, 2022 •
edited

In the Jupyter docs I noticed that the in-page {contents} directive was producing some slightly off output. So this PR updates the contents directive to be styled more consistently with our other directives.

Here's what the in-page contents looks like now:

chrome_ygPYoT6afq

@choldgraf choldgraf force-pushed the contents branch 3 times, most recently from 30e884a to 2173cb1 Compare July 5, 2022 09:26
@choldgraf choldgraf changed the title Updating topic style Updating topic and contents directive style Jul 5, 2022
@choldgraf choldgraf added this to the 0.10 milestone Jul 6, 2022
@choldgraf
Copy link
Collaborator Author

I'll plan to merge this one in since I think it's pretty uncontroversial and improves an element we don't really have styling for right now

@choldgraf choldgraf changed the title Updating topic and contents directive style Update topic and contents directive style Jul 7, 2022
@choldgraf choldgraf merged commit 2718f5c into pydata:main Jul 7, 2022
@choldgraf choldgraf deleted the contents branch July 7, 2022 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.10
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@choldgraf