Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Issues
  • #22175
Closed
Open
Issue created Mar 13, 2017 by Administrator@rootContributor

Stop using hex values for form state and alert colors

Created by: oktalk

Color definitions for form states and alerts should derive from the 'general variable structure' where all other colors are defined.

As often as we can, we should follow our convention of defining values off of our base variables. So for instance, state-success, state-info, state-warning, etc. should all modify the base value of brand-success, brand-info, brand-warning, etc. To ovoid creating new colors.

The Bootstrap framework should take the position of not assuming too much. Bootstrap should not assume that your form state and alert colors would differ from your established brand colors (as most instances they wouldn't). This would lighten the initial configuration of Bootstrap.

The following is a bad practice. It defines new colors outside of our main color definitions, and makes configuration more cumbersome:

$state-success-text:             #3c763d !default;
$state-success-bg:               #dff0d8 !default;
$state-success-border:           darken($state-success-bg, 5%) !default;

Instead we should have something like the following:

$state-success-text:             invert($brand-success) !default;
$state-success-bg:               darken($brand-success, 10) !default;
$state-success-border:           darken($state-success-bg, 5%) !default;
Assignee
Assign to
Time tracking