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

Synchronize border and color classes with BS 5.3.2, add Color and background helper, behavior improvements #1005

Conversation

vrozkovec
Copy link
Contributor

No description provided.

@vrozkovec vrozkovec changed the title Synchronize border and color classes with BS 5.3.2 Synchronize border and color classes with BS 5.3.2, add Color and background helper Nov 24, 2023
@vrozkovec vrozkovec changed the title Synchronize border and color classes with BS 5.3.2, add Color and background helper Synchronize border and color classes with BS 5.3.2, add Color and background helper, behavior improvements Nov 25, 2023
@martin-g martin-g merged commit 73aa879 into martin-g:wicket-10.x-bootstrap-5.x Nov 27, 2023
2 checks passed
@martin-g
Copy link
Owner

Thank you, @vrozkovec !

@solomax
Copy link
Contributor

solomax commented Nov 27, 2023

@martin-g can you please cherry-pick this into wicket-9.x-bootstrap-5.x? :)

martin-g pushed a commit that referenced this pull request Nov 27, 2023
…kground helper, behavior improvements (#1005)

* Fix border classes, add new colors for BS5

* Align colors with BS 5

https://getbootstrap.com/docs/5.3/utilities/colors/

* Javadoc fix

* Add new implementation to accommodate Color and background helper

https://getbootstrap.com/docs/5.3/helpers/color-background/

* Allow BorderBehavior to use IModel, add Width enum

* ColorBehavior can accept IModel<Color>

* BackgroundColorBehavior now accepts IModel<Color>

* ColorAndBackgroundBehavior now accepts IModel<Color>

* Add getters and setters for color

(cherry picked from commit 73aa879)
@martin-g
Copy link
Owner

Done! Thanks for the reminder, @solomax !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants