-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Change easymotion decoration colors to use searchHighlight colors #1431
Conversation
fixes #1245 |
…sible to trigger though
@@ -294,8 +324,6 @@ export class EasyMotion { | |||
// Get keys after the depth we're at | |||
var keystroke = marker.name.substr(this.accumulation.length); | |||
|
|||
let fontColor = keystroke.length > 1 ? "orange" : "red"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am disappointed to take this out but am not sure how else I could accomplish it and still have it be readable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
decided to leave it in, and make fontcolor and background color change with a configuration flag instead
…ion option so that default behavior remains the same
I can get the same result without the library if we want, but it definitely cleans up the code
Also, I spent a while using different functions of it to manipulate the color wheel to try out different things for readability. Complementary font color, etc don't look good.
Black and white and using the light()/dark() luminosity formula seemed to give the best results.