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

voteEnhancement post score color option #1313

Closed
wants to merge 5 commits into from

Conversation

matheod
Copy link
Collaborator

@matheod matheod commented Jun 24, 2014

  • Default color are ... horrible :D
  • Possible you prefer to use len-i instead of reverse direction for.

@jewel-andraia jewel-andraia changed the title [WIP] Allow the user to choose is own coloration for Link [WIP] voteEnhancement post color method option Jun 24, 2014
@jewel-andraia jewel-andraia changed the title [WIP] voteEnhancement post color method option [WIP] voteEnhancement post color - option for different algorithm Jun 24, 2014
@jewel-andraia jewel-andraia changed the title [WIP] voteEnhancement post color - option for different algorithm [WIP] voteEnhancement post score color option Jun 24, 2014
@jewel-andraia
Copy link
Collaborator

TESTING NOTES: because colorLinkScore has changed from type=boolean to type=enum, you will need to manually reset that option's value. This is only a concern for people running dev builds.

@matheod
Copy link
Collaborator Author

matheod commented Jun 24, 2014

Yea that why I didn't make a migrate thing ^^

@kevinji
Copy link
Collaborator

kevinji commented Jun 24, 2014

Or you could just clear localStorage.

@jewel-andraia
Copy link
Collaborator

or #!settings/troubleShooter/resetToFactory or RESStorage remove RESoptions.voteEnhancements

@honestbleeps
Copy link
Owner

merged manually

@jewel-andraia jewel-andraia mentioned this pull request Jul 2, 2014
@jewel-andraia jewel-andraia changed the title [WIP] voteEnhancement post score color option voteEnhancement post score color option Jul 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants