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

Hide context around collapsed editable sections #41

Merged
merged 1 commit into from
May 9, 2024

Conversation

emesterhazy
Copy link
Collaborator

@emesterhazy emesterhazy commented May 5, 2024

This makes the file view tidier by hiding context sections which aren't
contiguous to at least editable sections that is expanded.

@emesterhazy emesterhazy changed the title Hide context around unexpanded editable sections Hide context around collapsed editable sections May 5, 2024
@emesterhazy emesterhazy force-pushed the push-lonqxpwnkpzy branch 2 times, most recently from 79e7fd0 to 1201ccd Compare May 6, 2024 16:03
@emesterhazy emesterhazy marked this pull request as ready for review May 6, 2024 16:03
tests/test_scm_record.rs Show resolved Hide resolved
src/ui.rs Outdated Show resolved Hide resolved
src/ui.rs Outdated Show resolved Hide resolved
src/ui.rs Outdated Show resolved Hide resolved
src/ui.rs Outdated Show resolved Hide resolved
@arxanas
Copy link
Owner

arxanas commented May 7, 2024

I submitted a collaborator invite for you and approved the PR. When review feedback is addressed, you can merge it yourself. Thanks 🙂

This makes the file view tidier by hiding context sections which aren't
contiguous to at least editable section that is expanded.
@emesterhazy emesterhazy merged commit 473eb23 into arxanas:main May 9, 2024
2 checks passed
@emesterhazy emesterhazy deleted the push-lonqxpwnkpzy branch May 9, 2024 18:12
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.

2 participants