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

Simple score coloration type for comments #1660

Merged
merged 12 commits into from
Oct 4, 2014

Conversation

erikdesjardins
Copy link
Collaborator

Related to #1304 and #1375.
Has provisions for future presets, just add a new option and preset colors.
I'm not sure if it should have a more descriptive name than simple coloration.

Also closed a random unclosed element and removed a few unnecessary commas.

@@ -72,7 +72,7 @@
<ul id="RESKnownFeatureRequests"><li style="color: red;">Loading...</li></ul>
</li>
</ol>
<span id="submittingFeature" class="submittingToEnhancementButton">I still want to submit a feature request!<span>
<span id="submittingFeature" class="submittingToEnhancementButton">I still want to submit a feature request!</span>
Copy link
Collaborator

Choose a reason for hiding this comment

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

oops, thanks

@jewel-andraia
Copy link
Collaborator

kinda do want a more descriptive name, like "Classic" or "Orangered/Periwinkle"

jewel-andraia added a commit that referenced this pull request Oct 4, 2014
Simple score coloration type for comments
@jewel-andraia jewel-andraia merged commit 82bf2c0 into honestbleeps:master Oct 4, 2014
jewel-andraia added a commit that referenced this pull request Oct 4, 2014
follow-up to #1660.  i'm open to other suggestions
@jewel-andraia
Copy link
Collaborator

kinda arbitrarily went with "reddit classic". now accepting other suggestions

@jewel-andraia jewel-andraia mentioned this pull request Oct 4, 2014
@erikdesjardins erikdesjardins deleted the random-fixes branch October 5, 2014 00:32
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

2 participants