eclipse-xtext / eclipse-xtext/xtext
Add overviewRuler support to the Xtext EmbeddedEditorFactory class
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 831
- Forks
- 330
- Avg merge
- 3d 7h
- Merged PRs (30d)
- 12
Description
Currently, the EmbeddedEditorFactory class does not support configuring the overview ruler, see EmbeddedEditorFactory.java#L244
This should be implemented if the caller configure the EmbeddedEditorFactory by calling e.g. a method like .showOverviewRuler() the overview ruler should be present on the embedded editor.
The following picture demonstrates the current (left side) and the desired (right side) behavior:
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in org.eclipse.xtext.ui/src/org/eclipse/xtext/ui/editor/embedded/EmbeddedEditorFactory.java around line 244 and inspect how the EmbeddedEditorFactory configures embedded editors. Trace the existing configuration methods and determine how an opt-in showOverviewRuler-style setting should be represented. Done means callers can enable the option and the embedded editor displays the overview ruler, while existing callers remain unchanged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- desktop
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100