Skip to content

Conversation

@ajdelaguila
Copy link
Contributor

What type of PR is this?

  • 🛠 Enhancement

Description

Adding support to highlight commented lines of YAML syntax.

Added tests?

  • 🙅 no, because I think that they aren't needed

Added to documentation?

  • 🙅 I think that this change does not require a documentation update

@ajdelaguila
Copy link
Contributor Author

@nausharipov I've created this new one after synchronising my own fork. If you something missing, please let me know.

@nausharipov nausharipov merged commit 5b001fa into akvelon:main Jul 28, 2025
1 of 2 checks passed
razankv13 added a commit to zingdata/flutter-code-editor that referenced this pull request Oct 20, 2025
This merge brings in important bug fixes and features from main while preserving
all autocomplete/suggestions improvements from improve-editor branch.

Changes from main integrated:
- Flutter/Dart version upgrade (3.22.2/3.4.3)
- Dispose fixes (akvelon#298): Added _disposed flag and mounted checks
- UndoHistoryController support (akvelon#302)
- Line height in default text style fix (akvelon#297)
- YAML support and comments (akvelon#305, akvelon#199)
- JS/TS single line comments (akvelon#291)
- Paired symbols support (akvelon#192)
- Line numbers alignment fix (akvelon#307)
- Pull request template
- Various dependency updates

Changes preserved from improve-editor:
- Refactored code_controller structure (directory with helpers)
- SQL formatter and suggestions logic
- Word insertion helper
- Popup positioning improvements
- Gutter scrollbar hiding
- Browser detection enhancements
- All autocomplete/suggestions improvements

Conflicts resolved:
- code_field.dart: Integrated dispose, undo, and line height fixes
- code_controller.dart: Kept refactored structure, added dispose fix
- gutter.dart: Kept scrollbar hiding feature
- popup.dart & popup_controller.dart: Kept positioning improvements
- parser_factory.dart: Added YAML support with package imports

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
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