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

alert boxes need to be restyled to match color scheme #14

Open
krancour opened this issue Mar 1, 2022 · 6 comments
Open

alert boxes need to be restyled to match color scheme #14

krancour opened this issue Mar 1, 2022 · 6 comments
Assignees
Labels
good first issue Good for newcomers
Milestone

Comments

@krancour
Copy link
Contributor

krancour commented Mar 1, 2022

Bootstrap "alert" boxes (primary variant is the only one we currently use) should be restyled to be more consistent with the color scheme used across all Brigade sites.

Something like this maybe:

$colorm: #37b0e7;
$colorl: #EFF4F5;

.alert-primary {
  border-color: $colorm !important;
  background-color: $colorl !important;
}
@krancour krancour added the good first issue Good for newcomers label Mar 1, 2022
@krancour krancour changed the title alert boxes alert boxes need to be restyled to match color scheme Mar 1, 2022
@coder-abdo
Copy link
Contributor

to change the color schemes in bootstrap you need to change it in the bootstrap.scss file and I think this file in the react-bootstrap sass file in the node_modules

@krancour
Copy link
Contributor Author

krancour commented Mar 2, 2022

@coder-abdo it is possible to override this in src/App.scss. Most everything in that file is an override of Bootstrap default styles.

@krancour krancour added this to the v0.2.0 milestone Mar 3, 2022
@DhairyaBahl
Copy link
Contributor

@krancour Kindly assign this to me. Would love to work on this as well.

@krancour
Copy link
Contributor Author

krancour commented Mar 8, 2022

Assigned

@krancour
Copy link
Contributor Author

krancour commented Mar 8, 2022

@DhairyaBahl actually, sorry... my bad. @coder-abdo was already working on this in #32.

This is a good time to remind people that if you open a PR, please do link to the issue it addresses so we can avoid duplicate work.

@krancour krancour assigned coder-abdo and unassigned DhairyaBahl Mar 8, 2022
@DhairyaBahl
Copy link
Contributor

@DhairyaBahl actually, sorry... my bad. @coder-abdo was already working on this in #32.

@krancour No issues😄.

@krancour krancour modified the milestones: v0.2.0, v0.3.0 Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants