Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Make search highlight more visible #32

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

evindor
Copy link

@evindor evindor commented Apr 4, 2014

The default one is very hard to notice. I guess i'm not the only one who thinks so.
Screenshot

@tomasr
Copy link
Owner

tomasr commented Apr 12, 2014

Sorry guys, I've been reluctant to merge changes to the console side as I don't use it, and can't always verify what the end results are. I'm going through all the pending ones now. So do you want this one or the alternative proposed?

@justinmk
Copy link
Contributor

I prefer @vellvisher 's suggestion. evindor@8435b84#commitcomment-5984725

@evindor
Copy link
Author

evindor commented Apr 12, 2014

@vellvisher's suggestion seems to be more solid as it's from original monokai, however i really liked the bright orange i made, great visibility. But i guess it's too extreme to be the default one.

tomasr added a commit that referenced this pull request Apr 12, 2014
@tomasr
Copy link
Owner

tomasr commented Apr 12, 2014

OK, done. @evindor submit a patch with a global setting for controlling it as an option, if you want, that way we can have the best of both worlds :)

pR0Ps pushed a commit to pR0Ps/molokai-dark that referenced this pull request Oct 29, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants