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

Selection history navigation with Alt+[ and Alt+] keys should filter out empty selections #2111

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

0SlowPoke0
Copy link
Contributor

No description provided.

@0SlowPoke0 0SlowPoke0 marked this pull request as ready for review November 17, 2024 21:07
@Keavon
Copy link
Member

Keavon commented Nov 21, 2024

!build

Copy link

📦 Build Complete for c480ae5
https://c4515b75.graphite.pages.dev

@Keavon
Copy link
Member

Keavon commented Nov 21, 2024

Unless I'm doing something wrong, as far as I can tell from my testing, I think this entirely breaks the whole feature? Did you test this yourself?

@Keavon Keavon marked this pull request as draft November 21, 2024 02:58
@0SlowPoke0
Copy link
Contributor Author

@Keavon Sorry, I made a mistake in step back function, can you please check the changes I made

@Keavon
Copy link
Member

Keavon commented Nov 24, 2024

I'll be able to check this in a week. Thanks for your patience.

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