Skip to content

Conversation

@MartijnCuppens
Copy link
Member

@MartijnCuppens MartijnCuppens commented Nov 22, 2019

Before: https://twbs-bootstrap.netlify.com/docs/4.3/forms/checks/

image

After: https://deploy-preview-29714--twbs-bootstrap.netlify.com/docs/4.3/forms/checks/

image

Works in Chrome, FF & Chromium Edge. The themed checkboxes don't apply to EdgeHTML & IE in v5, these will have the native fallback.

@MartijnCuppens MartijnCuppens requested a review from a team as a code owner November 22, 2019 16:12
@MartijnCuppens MartijnCuppens changed the title Keep themed appearance for print Keep themed appearance for checkboxes when page is printed Nov 22, 2019
Copy link
Member

@mdo mdo left a comment

Choose a reason for hiding this comment

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

Oh shit what, that's awesome.

@MartijnCuppens
Copy link
Member Author

We'll need to backport this manually to v4-dev. And we'll have to find a solution for IE in v4 (I'll keep #28483 open to remind us)

@MartijnCuppens MartijnCuppens merged commit 460ba06 into master Nov 24, 2019
@MartijnCuppens MartijnCuppens deleted the master-mc-checkbox-and-radio-print-fixes branch November 24, 2019 19:01
mdo added a commit that referenced this pull request Sep 29, 2020
Keeps checkboxes, radios, and switches with their intended design when printing.

Backports #29714
@mdo mdo mentioned this pull request Sep 29, 2020
XhmikosR pushed a commit that referenced this pull request Sep 30, 2020
Keeps checkboxes, radios, and switches with their intended design when printing.

Backports #29714
olsza pushed a commit to olsza/bootstrap that referenced this pull request Oct 3, 2020
mdo added a commit that referenced this pull request Oct 4, 2020
Keeps checkboxes, radios, and switches with their intended design when printing.

Backports #29714
XhmikosR pushed a commit that referenced this pull request Oct 5, 2020
Keeps checkboxes, radios, and switches with their intended design when printing.

Backports #29714
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.

3 participants