Skip to content

Conversation

@Bogdan-Belogurov
Copy link
Contributor

@Bogdan-Belogurov Bogdan-Belogurov commented Jun 8, 2025

Description

Add EditorCommands menu and update EditorInstance to be able to move lines up and down

Related Issues

Checklist

  • I read and understood the contributing guide as well as the code of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

Screen.Recording.2025-06-08.at.09.04.02.mov

Update `EditorInstance` to be able to move lines up and down
@thecoolwinter
Copy link
Collaborator

Looks like this methods may need to be marked public

@Bogdan-Belogurov
Copy link
Contributor Author

@thecoolwinter Hi! Do you mean moveLinesUp and moveLinesDown methods?
Cause I use these methods only in the CodeEdit module 🤔

And I noticed an issue in the current solution.

thecoolwinter added a commit to CodeEditApp/CodeEditSourceEditor that referenced this pull request Jun 25, 2025
### Description

Guarantees that all text coordinator's lifetimes match that of the
editor's.

### Related Issues

* CodeEditApp/CodeEdit#2058

### Checklist

- [x] I read and understood the [contributing
guide](https://github.com/CodeEditApp/CodeEdit/blob/main/CONTRIBUTING.md)
as well as the [code of
conduct](https://github.com/CodeEditApp/CodeEdit/blob/main/CODE_OF_CONDUCT.md)
- [x] The issues this PR addresses are related to each other
- [x] My changes generate no new warnings
- [x] My code builds and runs on my machine
- [x] My changes are all related to the related issue above
- [x] I documented my code

### Screenshots
@thecoolwinter
Copy link
Collaborator

I'll be updating CE to use the latest version of CESE which will unblock this pr. I'll follow up after

@thecoolwinter
Copy link
Collaborator

Merged the changes from main and this works now. Great stuff, and good catch on the coordinator lifecycle bug,

@austincondiff austincondiff merged commit 097a685 into CodeEditApp:main Jul 9, 2025
2 checks passed
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.

3 participants