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

add support for toggling line numbers on and off #124

Conversation

ben-p-commits
Copy link
Contributor

Description

Adds support for #115, ✨Option To Hide Line Numbers. Simply shows/hides ruler view. should be easy enough in the future to include affordances for enabling/disabling code folding ribbon, code review, other rulerview utilities once they come into play.

Demo

line-numbers

@austincondiff
Copy link
Collaborator

austincondiff commented Jan 23, 2023

Should we also shift all content to the left when the line numbers are hidden? Is the space the line numbers were consuming not necessary anymore?

It looks like Xcode does this...

Screen.Recording.2023-01-23.at.4.40.13.PM.mov

@ben-p-commits
Copy link
Contributor Author

@austincondiff - ooh okay, i'll see what I can do

@ben-p-commits
Copy link
Contributor Author

ben-p-commits commented Jan 24, 2023

Update: created a PR over in STTextView to enable this behavior without introducing code smell in CETV...

krzyzanowskim/STTextView#11

@austincondiff - if we can get that change approved: we get the improved behavior:

hide-rulers.mp4

@lukepistrol
Copy link
Member

@ben-p-commits Ok lets wait for the outcome of the PR on STTextView for now.

@lukepistrol lukepistrol marked this pull request as draft January 25, 2023 09:43
@lukepistrol lukepistrol added the blocking There is a dependency that needs updating first label Jan 25, 2023
@ben-p-commits
Copy link
Contributor Author

@lukepistrol, closing this in light of #127

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocking There is a dependency that needs updating first
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants