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

Bootstrap 5 #332

Closed
vitorsr opened this issue May 7, 2021 · 6 comments · Fixed by #677
Closed

Bootstrap 5 #332

vitorsr opened this issue May 7, 2021 · 6 comments · Fixed by #677
Labels
enhancement New feature or request needs: upstream Needs action from an upstream repository.

Comments

@vitorsr
Copy link

vitorsr commented May 7, 2021

I know it’s early to point out but I figured it’d be interesting to open an issue to track upcoming development on supporting or not Bootstrap 5 [1, 2].

[1] https://blog.getbootstrap.com/2021/05/05/bootstrap-5
[2] https://getbootstrap.com/docs/5.0/migration

@vitorsr vitorsr added the enhancement New feature or request label May 7, 2021
@welcome
Copy link

welcome bot commented May 7, 2021

Thanks for opening your first issue here! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out EBP's Code of Conduct. Also, please try to follow the issue template as it helps other community members to contribute more effectively.

If your issue is a feature request, others may react to it, to raise its prominence (see Feature Voting).

Welcome to the EBP community! 🎉

@choldgraf
Copy link
Member

For sure - though right now we are inheriting from there pydata theme, so unless that changes we will get whatever bootstrap version from them

@choldgraf choldgraf added needs: upstream Needs action from an upstream repository. enhancement New feature or request and removed enhancement New feature or request labels May 7, 2021
@vitorsr
Copy link
Author

vitorsr commented May 7, 2021

Thanks for the prompt response - I wasn’t aware of this dependency. (I thought the book theme was just inspired by pydata’s.)

On a side note I am primarily concerned with properly displaying math content on both mobile and desktop. The pydata theme lends itself to some - in my opinion - questionable practices that reverberate on math elements (e.g., overflow). I was happy to see that it was not the case in the book theme.

pydata book
pydata book

However there are still some minor issues such as word wrap (e.g., see [1] on mobile) that I believe are amenable to being solved by Bootstrap [2].

[1] https://sphinx-book-theme.readthedocs.io/en/latest/reference/demo.html#id10
[2] https://getbootstrap.com/docs/5.0/utilities/text/#word-break

@pradyunsg
Copy link
Member

The pydata theme lends itself to some - in my opinion - questionable practices that reverberate on math elements (e.g., overflow).

I'm pretty sure this would be considered a bug that they'd like to fix. :)

@pradyunsg
Copy link
Member

However there are still some minor issues such as word wrap (e.g., see [1] on mobile) that I believe are amenable to being solved by Bootstrap [2].

FWIW, you don't need to upgrade to Bootstrap 5 for this.

It's literally setting:

.text-wrap {
    word-wrap: break-word;
    word-break: break-word;
}

Which, we can do here as well. I'm pretty sure the main thing that'll have these issues is links.

@choldgraf
Copy link
Member

Indeed it would be great to see this overflow fizz on the pydata theme and the reword fix here or in the pydata theme 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs: upstream Needs action from an upstream repository.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants