search

zen0wu / topcoder-greed

greedy editor for topcoder arena
Apache License 2.0
229 stars 48 forks source link

Sane defaults for problem statement #119

Closed ashashwat closed 10 years ago

ashashwat commented 10 years ago

From current default.conf,

# set One of these to true to enable the theme:
colorThemeBlack       = true
colorThemeBlue        = false
colorThemeLowContrast = false

The default color theme [black].

screen shot 2014-01-04 at 5 51 46 am

The color theme from the original 2.0 beta.

screen shot 2014-01-04 at 5 51 55 am

The colortheme Black is the default which IMHO is extremely hard on eyes. I know it is just a matter of changing true to false in the config and readability is a matter of personnel preference. Is it really ok to to use retro style as default instead of the original beta one ?

zen0wu commented 10 years ago

Yes, I agree with you that the black is too much. We're planning to refactor the problem statement template in #117, this will be one of the issues.

zen0wu commented 10 years ago

How about now? The color has been changed. You can try out the latest release version.

ashashwat commented 10 years ago

Looks great. Just configured the latest version. We have 2 conf files now ?

zen0wu commented 10 years ago

To the user, it's just one conf, greed.conf. It's because the default.conf is too big so we split it, to the users, it's the same as before.