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

Added editor rulers #1709

Merged
merged 7 commits into from
Mar 22, 2018
Merged

Added editor rulers #1709

merged 7 commits into from
Mar 22, 2018

Conversation

rayou
Copy link
Contributor

@rayou rayou commented Mar 21, 2018

Feature requested in #1645

Rulers is disabled by default.

I will add unit tests once @nlopin adds jest package into his commits.

mar-21-2018 22-40-51
Note: Ruler 120 was hidden due to small viewport.

@@ -57,6 +57,10 @@
opacity: 1 !important;
pointer-events: auto !important;
}
.CodeMirror-ruler {
border-left-color: rgba(142, 142, 142, 0.5);
mix-blend-mode: difference;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

overwrite the default CodeMirror Ruler color.

The default ruler color is #ccc which is too bright in dark themes, so I made this tweak to ensure the ruler fits all kind of themes.

@Rokt33r Rokt33r added the awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. label Mar 22, 2018
@Rokt33r Rokt33r added next release (v0.11.3) and removed awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. labels Mar 22, 2018
@Rokt33r Rokt33r merged commit b766b08 into BoostIO:master Mar 22, 2018
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