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

border color issues caused by $color generated properties that conflict in some rules. #1185

Closed
the-j0k3r opened this issue Jul 18, 2020 · 3 comments
Labels
bug 🐛 Something isn't working

Comments

@the-j0k3r
Copy link
Member

Capture

"$color: #444d56": "#afafaf", // .full-commit .commit-desc pre
is the guilty rule

its generating a bright border for .Box--overlay this bright VALUE is OK for some other rules but NOT this ones border.

  /* github-mobile: "#fff", "#444d56", "rgba(0,0,0,.4)" */
  body[class="page-responsive"] .Box--overlay {
    background-color: #151515;
    border-color: #afafaf;
    box-shadow: 0 0 18px #000;
  }

@silverwind Im out of ideas, $color is more trouble than its worth IMO. Unless its A LOT smarter

@the-j0k3r
Copy link
Member Author

Capture

and so on goes this song

@the-j0k3r
Copy link
Member Author

This also affects border for https://github.com/bus-stop/terminus/pull/104/checks?check_run_id=885718952 for .CheckRun-header

Capture

@the-j0k3r the-j0k3r changed the title border color issues border color issues caused by $color generated properties that conflict in some rules. Jul 21, 2020
@silverwind
Copy link
Member

There ought to be issues like that but generally I like to strive for a inversion or close to it, but it'll not work on all cases and we should just override those specific issues.

silverwind added a commit that referenced this issue Jul 21, 2020
* add .flash-warn border to generator, fixes #1188 (silverwind)
* change foreground of warning toast icon, fixes #1189 (silverwind)
* override generated dark borders - fixes #1185 (silverwind)
* handle popover arrows in generator (silverwind)
silverwind added a commit that referenced this issue Jul 21, 2020
silverwind added a commit that referenced this issue Jul 21, 2020
* better fix for #1185 (silverwind)
* restore semi-transparent header effect (silverwind)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants