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

Closes #1122 Add callout variants for all theme colors. #1123

Merged
merged 8 commits into from
Feb 1, 2024
Merged

Conversation

joeparsons
Copy link
Member

@joeparsons joeparsons commented Jan 24, 2024

Closes #1122

Review site: https://review.digital.arizona.edu/arizona-bootstrap/issue/1122/docs/5.0/components/callouts/

Screenshot 2024-01-31 at 10 29 31 AM

How to test

Verify Standard CKEditor can accept and appropriately display the following classes on a div.

.callout-leaf
.callout-sky
.callout-info
.callout-light
.callout-success
.callout-warning

@joeparsons joeparsons added enhancement New feature or request backport Backport to 2.x labels Jan 24, 2024
@joeparsons joeparsons self-assigned this Jan 24, 2024
@joeparsons joeparsons removed the backport Backport to 2.x label Jan 24, 2024
@joeparsons joeparsons marked this pull request as ready for review January 25, 2024 18:06
@joeparsons joeparsons requested a review from a team January 25, 2024 18:13
mmunro-ltrr
mmunro-ltrr previously approved these changes Jan 25, 2024
Copy link
Member

@mmunro-ltrr mmunro-ltrr left a comment

Choose a reason for hiding this comment

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

The documentation on the main branch needs an overhaul to align it with upstream Bootstrap v5, but that's a different issue. One relevant section there is
https://getbootstrap.com/docs/5.3/customize/components/#creating-your-own
Regenerating the package-lock.json yet again was necessary to get a local lint check to pass.

danahertzberg
danahertzberg previously approved these changes Jan 26, 2024
trackleft
trackleft previously approved these changes Jan 26, 2024
scss/_callout.scss Outdated Show resolved Hide resolved
Copy link
Member

@mmunro-ltrr mmunro-ltrr left a comment

Choose a reason for hiding this comment

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

Re-checked on a local build.

@trackleft trackleft merged commit 31c8542 into main Feb 1, 2024
2 checks passed
@trackleft trackleft deleted the issue/1122 branch February 1, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow "warning", "danger", etc class on all elements, just like UA Bootstrap allows
5 participants