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

🩹(frontend) define SASS variable color out with bootstrap v5 #197

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

carofun
Copy link
Contributor

@carofun carofun commented Jun 9, 2021

Purpose

Fix different style regression caused by the upgrade of bootstrap v5
$table-head-color used in saas fil wasn't set-up as it was defined in bootstrap variable.
float-left, or float-right are not defined anymore,
links need to be fixed
Capture d’écran 2021-06-10 à 09 54 15
Capture d’écran 2021-06-10 à 09 54 05

Proposal

nb : 1 commit
Follow documentation and fix miss out classes
https://getbootstrap.com/docs/5.0/migration/

  • Defining $table-head-color:#495057; color that it's set on now with this no assignement. Keeping this variable is better if we want to change it's value later on.

Capture d’écran 2021-06-09 à 16 33 17
Capture d’écran 2021-06-09 à 16 33 11
Capture d’écran 2021-06-09 à 16 38 59

  • fixing links
    ... to be continued

@carofun carofun self-assigned this Jun 9, 2021
@carofun carofun requested a review from madmatah June 9, 2021 14:39
@carofun carofun changed the title 🩹(frontend) define SASS color variable 🩹(frontend) define SASS variable color out with bootstrap v5 Jun 9, 2021
@carofun carofun force-pushed the fix/renovate-sass branch 2 times, most recently from b2a713e to 3fb50b2 Compare June 10, 2021 07:24
@carofun carofun added WIP and removed needs review labels Jun 10, 2021
Bootstrap version 5 is out. We want Ashley to be updated with
latest versions of our dependencies.
$table-head-color used in saas fil is not set-up anymore. The upgrade of the
library bootrstrap version 4 to 5 doesn't have this variable pre-configured
anymore. As a result, the frontend won't be able to be build anymore. We fix
this issue by giving a color to this variable so that Ashley can use bootstrap
5.
@carofun carofun linked an issue Jun 10, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to bootstrap v5
1 participant