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

The Orc eclipse plugin does not highlight matching braces #191

Open
arthurp opened this issue Feb 18, 2017 · 7 comments
Open

The Orc eclipse plugin does not highlight matching braces #191

arthurp opened this issue Feb 18, 2017 · 7 comments

Comments

@arthurp
Copy link
Member

arthurp commented Feb 18, 2017

Unlike the Scala/Java editors and the default eclipse text editor, the Orc editor does not highlight the matching brace of the brace before the cursor. This is a small issue, but is a surprisingly important feature for me (since I lose track of brace matching easily).

Is this an easy fix? Can we just turn it on and provide a list of braces? Or would be have to implement it?

@arthurp
Copy link
Member Author

arthurp commented Feb 18, 2017

@jthywiss You would be the one who might actually know what would go into making this happen. I'd like to address this if it will take less than a few hours of work since I've realized that improving the plugin is important for me now that I am actually programming in Orc instead of working on the Orc compiler/interpreter.

@jthywiss
Copy link
Member

This is implemented in the master head, and I see it functioning in an open Orc editor right now for parentheses, braces, and brackets.

Do you have the master head plug-in? Do your Eclipse prefs have this enabled? Are there more pair types we need to match?

@arthurp
Copy link
Member Author

arthurp commented Feb 21, 2017

I have the a recent build of the plug-in. I built it myself at the end of last week from master.

Highlighting is working on both Scala and Java and I cannot find any option that is specific to Orc or any related mode.

Highlighting is currently not working on any kind of brace. We might need to add some kinds later, but for the moment I just want to have it for () and {}.

@jthywiss jthywiss changed the title The Orc eclipse plugin does not matching braces The Orc eclipse plugin does not highlight matching braces Feb 21, 2017
@jthywiss
Copy link
Member

Perhaps we are talking about different functions.

The standard Eclipse source editor highlights the matching character of a pair when your insertion point is next to the start or end of a pair. By default this is a gray outline. This runs in the Orc editor. See OrcEditor.java lines 103–106 for the implementation.

The JDT editor extends several other editor functions involving braces, such as the double-click strategy, folding, and expanding selections. Those are not implemented in the Orc editor, but the double-click strategy enhancement, for instance, would be easy.

@arthurp
Copy link
Member Author

arthurp commented Feb 21, 2017 via email

@jthywiss
Copy link
Member

Eclipse IDE for Eclipse Committers

Version: Neon.2 Release (4.6.2)
Build id: 20161208-0600

@arthurp
Copy link
Member Author

arthurp commented Feb 21, 2017 via email

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

No branches or pull requests

2 participants