Skip to content

Conversation

@mdo
Copy link
Member

@mdo mdo commented Feb 21, 2019

Removes the print styles, print utilities, and more. Far too many bugs around this stuff for us to keep maintaining as so much of it depends on the exact page design and layout.

@mdo mdo requested a review from a team as a code owner February 21, 2019 20:54
@XhmikosR XhmikosR changed the title v5: Drop all print styles and classes [WIP] v5: Drop all print styles and classes Feb 22, 2019
@MartijnCuppens
Copy link
Member

I wouldn't drop .d-print-none, that's a pretty handy utility class.

@mdo
Copy link
Member Author

mdo commented May 8, 2019

Should be ready for review now, @twbs/css-review.

@mdo mdo changed the title [WIP] v5: Drop all print styles and classes v5: Drop all print styles and classes May 8, 2019
@mdo mdo requested a review from a team May 8, 2019 18:17
@XhmikosR
Copy link
Member

XhmikosR commented May 8, 2019

So, just a last comment. Are we 100% sure about this? I mean, if you just do a print preview of our docs and compare you will see how bad it is now.

@MartijnCuppens
Copy link
Member

It's actually better here (Chrome, mac) and on Firefox it's equally bad. Our print styles fix some things while introducing other issues. We might have a look at reintroducing print style sheets or optimizing our docs if we know for sure we don't break anything else.

@MartijnCuppens MartijnCuppens merged commit 473689c into master May 9, 2019
@MartijnCuppens MartijnCuppens deleted the v5-drop-print branch May 9, 2019 05:58
@SPodjasek SPodjasek mentioned this pull request Nov 23, 2022
3 tasks
crhistianramirez pushed a commit to crhistianramirez/headstart that referenced this pull request Dec 28, 2022
Removed print styles and $enable-print-styles variable. Print display classes are still around. See twbs/bootstrap#28339
HmRakib14 pushed a commit to HmRakib14/headstart that referenced this pull request May 22, 2025
Removed print styles and $enable-print-styles variable. Print display classes are still around. See twbs/bootstrap#28339
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 this pull request may close these issues.

4 participants