Skip to content

Commit

Permalink
Use Vim colors in white terminal
Browse files Browse the repository at this point in the history 
The highlighting is now exactly like in Vim in a white background terminal.
  • Loading branch information
@certik
certik committed Jul 28, 2010
1 parent 074844b commit 38fed50
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions pudb/theme.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -80,21 +80,21 @@ def add_setting(color, setting):
("dialog title", add_setting("white", "bold"), "dark cyan"),

# highlighting
("source", "yellow", "dark blue"),
("source", "black", "default"),
("focused source", "black", "dark green"),
("highlighted source", "black", "dark magenta"),
("current source", "black", "dark cyan"),
("current focused source", "white", "dark cyan"),
("current highlighted source", "white", "dark cyan"),

("keyword", add_setting("white", "bold"), "dark blue"),
("kw_namespace", add_setting("white", "bold"), "dark blue"),
("literal", "light magenta", "dark blue"),
("string", "light magenta", "dark blue"),
("punctuation", "light gray", "dark blue"),
("comment", "light gray", "dark blue"),
("classname", "yellow", "dark blue"),
("funcname", "yellow", "dark blue"),
("keyword", "brown", "default"),
("kw_namespace", "dark magenta", "default"),
("literal", "black", "default"),
("string", "dark red", "default"),
("punctuation", "black", "default"),
("comment", "dark blue", "default"),
("classname", "dark cyan", "default"),
("funcname", "dark cyan", "default"),

]

7 comments on commit 38fed50

@asmeurer
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm. My terminal is green text on a black background, so your mods don't work for me :). But I think I can use this to make the text the same colors as my text editor.

Let's keep each other updated on any other mods we make!

@certik
Copy link
Owner Author

@certik certik commented on 38fed50 Jul 28, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure. Just use the previous patch, where I export some more Python syntax features, and modify this patch. We should create more themes for pudb and allow the user to switch.

In any case, I hate the default, as it looks like this old "Turbo Pascal" colors:

Turbo Pascal

later versions even had colors:

Turbo Pascal

and github comments rock (you should view this online).

@asmeurer
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha ha! Awesome. I knew you could do links, formatting, and code blocks, but I never knew you could do that.

Yeah, it is like some kind of old Windows thing. I am definitely going to make it look better now that you've inspired me.

By the way, I don't think Andreas Kloeckner (the original author of PuDB) knows about our forks. If one of us ends ups implementing the themes, we should email him and see if he will integrate it into his official repo.

@certik
Copy link
Owner Author

@certik certik commented on 38fed50 Jul 28, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's actually an old DOS thing, it has nothing to do with Windows. The first screenshot is what I was using in 1990, when I was learning how to program.
The first program actually happens to apply to me quite well, I loved TP, but now I can't understand what I liked on it.

As to Andreas, I'll let him now, we had couple beers in Miami (I think it was Miami, maybe it was some other conference).

@asmeurer
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my colors branch. It now looks like this for me, which is just like the Midnight Theme in XCode. I also got rid of the white on cyan, which was killing my eyes.

Aaron's Colors

@certik
Copy link
Owner Author

@certik certik commented on 38fed50 Jul 28, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool. We should have this as a theme and my thing as another theme, and the original one as yet another theme.

@inducer
Copy link
Contributor

@inducer inducer commented on 38fed50 Aug 2, 2010

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool. Please do implement themes--this was the original intention, but I never had any incentive to put in the actual theme functionality, since I actually like the Turbo Pascal look. :) If you need help, please let me know.

Andreas

Please sign in to comment.