Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Use a consistent border color - #3657

Merged
fregante merged 3 commits into
refined-github:masterfrom
ocarreterom:update-gray-border
Nov 1, 2020
Merged

Use a consistent border color#3657
fregante merged 3 commits into
refined-github:masterfrom
ocarreterom:update-gray-border

Conversation

@ocarreterom

@ocarreterom ocarreterom commented Oct 15, 2020

Copy link
Copy Markdown
  • move border color to CSS variable
  • normalize gray border color to #e1e4e8

Thanks for contributing! 馃崉

  1. LINKED ISSUES:
    No issue

  2. TEST URLS:
    https://github.com/ (dashboard)

  3. SCREENSHOT:
    The border after "All activity" was darker.

Before After
Captura de pantalla 2020-10-16 a las 1 07 09 Captura de pantalla 2020-10-16 a las 1 06 46

- move to CSS variable
- normalize to color `#e1e4e8`
- use 1px border width for pinned issues
Reset pinned issues border to 2px after read the discussion 馃槃
#3491
Reset border width to 2px since is not the purpose of this PR.
@yakov116

Copy link
Copy Markdown
Member

Can you please provide before and after screenshots

@ocarreterom

Copy link
Copy Markdown
Author

Can you please provide before and after screenshots

Done! It's a subtle change 馃槄

@fregante

Copy link
Copy Markdown
Member

I have to double check but I think that works! Thank you!

@fregante fregante changed the title Update gray border Use a consistent border color Oct 16, 2020
@fregante
fregante merged commit 52978fb into refined-github:master Nov 1, 2020
@fregante

fregante commented Nov 1, 2020

Copy link
Copy Markdown
Member

cc @silverwind heads up for updated CSS colors

@silverwind

Copy link
Copy Markdown
Contributor

Should probably be automatically handled by our tooling. BTW GitHub now has CSS vars of their own, you should switch to them (for example --color-border-primary for your border).

@ocarreterom
ocarreterom deleted the update-gray-border branch November 1, 2020 19:03
@fregante fregante mentioned this pull request Nov 1, 2020
@silverwind

Copy link
Copy Markdown
Contributor

Automation picked it up just fine: StylishThemes/GitHub-Dark@badb3c9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants