-
Notifications
You must be signed in to change notification settings - Fork 65
Upgrade to Bootstrap v5.3.1 #749
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
Merged
Merged
Changes from all commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
83c260d
Bump Bootstrap & Bootswatch to 5.3.1
cpsievert 913bda9
Update vendor prefix whitelist; only keep Bootswatch source; reduce n…
cpsievert 039e0d8
Temporarily remove problematic patches
cpsievert 5693783
Add in missing variables-dark import
cpsievert 9f11f8f
Check-in initial changes to Bootstrap/Bootswatch
cpsievert b9dabb1
Reflect changes to Bootstrap's color-contrast() in bslib's version
cpsievert d4b5bc3
Render 033-accordion-defaults.patch obsolete by reflecting the change…
cpsievert 3f6e9de
Update 023-shiny-forms patch
cpsievert f1bd73f
Change well-bg's default to a value we can compute on (card-cap-bg is…
cpsievert a88f45f
Update 011-component-color-contrast-bs5 patch
cpsievert 41ba17e
Update 021-add-navbar-bg-vars-bs5 patch
cpsievert be7b882
Update 006-bootswatch-materia-bs5 patch
cpsievert 5f388d2
Update 006-bootswatch-superhero-bs5 patch
cpsievert e28ab0f
Update 006-bootswatch-spacelab-bs5 patch
cpsievert 1bd5ce5
Update 006-bootswatch-slate-bs5 patch
cpsievert d39d6bf
Update 006-bootswatch-darkly-bs5 patch
cpsievert 4e11d45
Update 006-bootswatch-cyborg-bs5 patch
cpsievert df6e1fe
Update 028-bsw-leverage-navbar-light-dark-bg patch
cpsievert 828c40f
Resave distributed files (GitHub Action)
cpsievert 4594843
Fix bad patch update to 006-bootswatch-cyborg-bs5
cpsievert eb66d73
Update test snapshots
cpsievert 525da1a
Use body-bg instead of input-bg to test contrasting since the latter …
cpsievert 7981f8c
`devtools::document()` (GitHub Actions)
cpsievert 1c3dd0d
Replace 027-navbar-active-color-bs5-legacy with more bs3compat rules
cpsievert 133b36b
Replace 027-navbar-active-color-bs5-legacy with more bs3compat rules
cpsievert 5d604e8
Resave distributed files (GitHub Action)
cpsievert cd04924
Fix according styling (null can't be used to set a new default value …
cpsievert fc67d8e
`devtools::document()` (GitHub Actions)
cpsievert 4e00d92
Fix subtle bug in 023-shiny-forms-bs5 patch
cpsievert 925a35c
Resave distributed files (GitHub Action)
cpsievert 90adbee
Resave data (GitHub Action)
cpsievert 4b8f520
Compatibility shim for selectize.js 0.12.4
cpsievert df2f596
Resave distributed files (GitHub Action)
cpsievert 725260b
Just always set the more careful selectize variable defaults
cpsievert 2969855
Update inst/bslib-scss/functions.scss
cpsievert File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| # DO NOT EDIT | ||
| # This file is auto-generated by tools/yarn_install.R | ||
| version_bs5 <- "5.2.2" | ||
| version_bs5 <- "5.3.1" | ||
| version_bs4 <- "4.6.0" | ||
| version_bs3 <- "3.4.1" | ||
| version_accessibility <- "1.0.6" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,19 @@ | ||
| // Cards are white by default, but we want a smarter default for .well | ||
| // that it appears gray by default | ||
| $well-bg: $card-cap-bg !default; | ||
| $well-bg: rgba($black, .03) !default; | ||
| $well-color: $body-color !default; | ||
|
|
||
| // Bootstrap 5.3.1 + selectize.js v0.12.4 (i.e., shiny v1.7.5) shim | ||
| // | ||
| // Starting with 5.3.x, Bootstrap started having Sass variables default to CSS | ||
| // variables (e.g., $input-color: var(--bs-body-color)), which is problematic for | ||
| // compuation that happens in Sass (e.g., `mix($color1, $color2)`). This bundle | ||
| // fixes this problem for shiny::selectizeInput()'s Sass code (future versions of | ||
| // selectizeInput() should hopefully address this problem, making this temporary | ||
| // stopgap solution no longer required). | ||
| $selectize-color-text: if(is-css-variable($input-color), $body-color, $input-color) !default; | ||
| $selectize-color-item: rgba($selectize-color-text, 0.1) !default; | ||
| $selectize-color-item-border: if(is-css-variable($input-border-color), $border-color, $input-border-color) !default; | ||
| $selectize-color-dropdown: if(is-css-variable($dropdown-bg), $body-bg, $dropdown-bg) !default; | ||
| $selectize-color-dropdown-border-top: rgba($input-border-color, 0.2) !default; | ||
| $selectize-color-dropdown-item-create-text: rgba($selectize-color-text, 0.5) !default; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1,5 @@ | ||
| // BS3 had .navbar { margin-bottom: 20px; } | ||
| $navbar-margin-bottom: 20px !default; | ||
|
|
||
| // BS5+ variable used for prefixing CSS vars | ||
| $prefix: bs- !default; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,18 @@ | ||
| // color-contrast() was introduced in Bootstrap 5. | ||
| // We include our own version for a few reasons: | ||
| // 1. Easily turn off warnings options(bslib.color_contrast_warnings=F) | ||
| // 2. Allow Bootstrap 3 & 4 to use color-contrast() in variable definitions | ||
| // 3. Allow Bootstrap 3 & 4 to use bs_get_contrast() | ||
|
|
||
| // Also note that color-contrast() lives in sass-utils since projects like Quarto | ||
| // and flexdashboard currently assume it exists there.... | ||
| @import "../sass-utils/color-contrast.scss"; | ||
|
|
||
|
|
||
| @function is-css-variable($x) { | ||
| @if (type-of($x) != string) { | ||
| @return false; | ||
| } @else { | ||
| @return str-slice($x, 1, 6) == 'var(--'; | ||
| } | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.