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

Set Org Name Text Color #393

Merged
merged 2 commits into from Aug 18, 2016
Merged

Set Org Name Text Color #393

merged 2 commits into from Aug 18, 2016

Conversation

geekgonecrazy
Copy link
Contributor

Sets it to look like:
image

Instead of:

image

@Mottie
Copy link
Member

Mottie commented Aug 18, 2016

Hi @geekgonecrazy!

Thanks for the contribution, but I think it might be better to add back some of our header selector color definitions (e.g. h1, h2,...) - what do you think @silverwind?

@xt0rted
Copy link
Member

xt0rted commented Aug 18, 2016

I'm running a fork of this in Edge and had to add in the following for the org page header:

h1.org-name {
    color: #fff !important;
}

When I ran into this I thought it was working in FireFox but looking just now shows it's not. Maybe it was changed recently.

@geekgonecrazy
Copy link
Contributor Author

Possibly 😄 That's the only one I had seen that was really bugging me in my daily workflow.

I saw some discussion going on in #388

@silverwind
Copy link
Member

I think it might be better to add back some of our header selector color definitions (e.g. h1, h2,...) - what do you think @silverwind?

I wouldn't. It's probably going to cause more issues in the long run to have different selectors than GitHub.

@geekgonecrazy care to move .org-name to #c0c0c0? It's our counterpart to GitHub's #333.

@geekgonecrazy
Copy link
Contributor Author

geekgonecrazy commented Aug 18, 2016

@silverwind done.

image

@Mottie Mottie merged commit c6aac1e into StylishThemes:master Aug 18, 2016
@Mottie
Copy link
Member

Mottie commented Aug 18, 2016

Thanks!

@geekgonecrazy geekgonecrazy deleted the patch-1 branch August 18, 2016 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants