Skip to content

Remove textmate grammar registration by delete key and select the next one for fast deletion#9438

Open
Chris2011 wants to merge 1 commit into
apache:masterfrom
Chris2011:feature/delete-on-keypress
Open

Remove textmate grammar registration by delete key and select the next one for fast deletion#9438
Chris2011 wants to merge 1 commit into
apache:masterfrom
Chris2011:feature/delete-on-keypress

Conversation

@Chris2011

Copy link
Copy Markdown
Contributor

If you have a veeeery long list of textmate grammar registrations in the options panel and you need to remove a line to check whether this makes the error or not, after removing with alt + r you need to select another one manually. This PR fixes this problem and also we can now delete via delete key.

PR approval and merge checklist:

  1. Was this PR correctly labeled, did the right tests run? When did they run?
  2. Is this PR squashed?
  3. Are author name / email address correct? Are co-authors correctly listed? Do the commit messages need updates?
  4. Does the PR title and description still fit after the Nth iteration? Is the description sufficient to appear in the release notes?

If this PR targets the delivery branch: don't merge. (full wiki article)

@Chris2011 Chris2011 added LSP [ci] enable Language Server Protocol tests ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Jun 12, 2026
@Chris2011 Chris2011 changed the title Remove by delete key and select the next one to faster deletion Remove textmate grammar registration by delete key and select the next one to faster deletion Jun 12, 2026
@Chris2011 Chris2011 force-pushed the feature/delete-on-keypress branch from 271eb9b to f43d0f0 Compare June 12, 2026 14:53
@Chris2011 Chris2011 changed the title Remove textmate grammar registration by delete key and select the next one to faster deletion Remove textmate grammar registration by delete key and select the next one for fast deletion Jun 12, 2026
@Chris2011 Chris2011 self-assigned this Jun 12, 2026
@mbien mbien added the UI User Interface label Jun 13, 2026
@mbien mbien added this to the NB31 milestone Jun 13, 2026

@matthiasblaesing matthiasblaesing left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only eyeballed this, but looks sane to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) LSP [ci] enable Language Server Protocol tests UI User Interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants