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

tango-dark theme: Changed caret color #29

Merged
merged 1 commit into from May 26, 2013
Merged

tango-dark theme: Changed caret color #29

merged 1 commit into from May 26, 2013

Conversation

pentix
Copy link
Contributor

@pentix pentix commented Apr 13, 2013

I saw several comments in the internet on changing the cursor color in the tango-dark theme. Sometimes, it was really difficult to find the cursor. I changed the cursor color to #ff0000 (red). It should be a lot easier to find the cursor now.

@KerSp66zQAx
Copy link

Matthew, looking the original code in https://github.com/barryvan/Geany-Tango-Dark/blob/master/geany-tango-dark.conf and the historý of this repo, the problem seems to be a simple typo; the caret should be white.

But note that changing it to white means to adapt brace_good and brace_bad too (at least; maybe others too).

@codebrainz codebrainz merged commit d5c8e33 into codebrainz:master May 26, 2013
@codebrainz
Copy link
Owner

I made some changes after this PR. Please test and offer alternative pull request if you don't like, or I can revert fa56dbe. I don't use this theme much so you guys tell me :)

@KerSp66zQAx
Copy link

It's a good improvement.

@KerSp66zQAx
Copy link

Mixed capitalization? :P

@codebrainz
Copy link
Owner

Darn I thought I got it all (I used Geany's colour tool), oh well :)

@pentix
Copy link
Contributor Author

pentix commented May 30, 2013

@codebrainz I like both! 👍
It's just important to find your caret :)

You don't have to revert anything.

codebrainz pushed a commit that referenced this pull request Jun 22, 2019
* Create README.md

* Update README.md

As per #30's new requirement
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

3 participants