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

Chore merge from c5f46b0 to 6d24ccc #2159

Merged
merged 28 commits into from
Jul 24, 2023

Conversation

julien-deramond
Copy link
Member

@julien-deramond julien-deramond commented Jul 23, 2023

@netlify
Copy link

netlify bot commented Jul 23, 2023

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 4e9cdb9
🔍 Latest deploy log https://app.netlify.com/sites/boosted/deploys/64be453b45636c00083b8f00
😎 Deploy Preview https://deploy-preview-2159--boosted.netlify.app/docs/5.3/components/collapse
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@julien-deramond julien-deramond changed the title Chore merge from c5f46b0 Chore merge from c5f46b0 to 6d24ccc Jul 24, 2023
@julien-deramond julien-deramond marked this pull request as ready for review July 24, 2023 05:33
Copy link
Member

@louismaximepiton louismaximepiton left a comment

Choose a reason for hiding this comment

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

Feels kind of weird to have custom tooltips/popovers with a violet bg even if it underlines the fact that we shouldn't use them. But I'm fine with it anyway.

Few comments to tak into account before merging!

site/content/docs/5.3/components/spinners.md Outdated Show resolved Hide resolved
site/content/docs/5.3/components/collapse.md Outdated Show resolved Hide resolved
@@ -220,17 +220,18 @@

// scss-docs-start custom-tooltip
.custom-tooltip {
--bs-tooltip-bg: var(--bs-primary);
--bs-tooltip-bg: var(--bd-violet-bg);
Copy link
Member

Choose a reason for hiding this comment

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

[Bootstrap side] It feels weird to have a reference to bd- inside our doc.

}
// scss-docs-end custom-tooltip

// scss-docs-start custom-popovers
.custom-popover {
--bs-popover-max-width: 12.5rem;
--bs-popover-border-color: var(--bs-primary);
--bs-popover-border-color: var(--bd-violet-bg);
--bs-popover-header-bg: var(--bd-violet-bg);
Copy link
Member

Choose a reason for hiding this comment

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

[Bootstrap side] It feels weird to have a reference to bd- inside our doc.

site/content/docs/5.3/migration.md Outdated Show resolved Hide resolved
Copy link
Member

@louismaximepiton louismaximepiton left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@julien-deramond julien-deramond merged commit c18f2ef into main Jul 24, 2023
13 checks passed
@julien-deramond julien-deramond deleted the main-jd-chore-merge-from-c5f46b0 branch July 24, 2023 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore merge Sync with Bootstrap
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants