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

auto-close for brackets #1041

Open
ghost opened this issue May 23, 2016 · 7 comments
Open

auto-close for brackets #1041

ghost opened this issue May 23, 2016 · 7 comments

Comments

@ghost
Copy link

ghost commented May 23, 2016

The auto-close brackets does work only for the first outer brackets (parenthesis,curly braces,square brackets), if one wants to have further brackets inside they are not auto-closed. It would be great if auto-closing where possible for any number of times and depth inside other same-brackets.
The following will happen if want 2 times brackets inside each other:

[[]
(()
{{}

it would be great if it could happen like this:

[[]]
(())
{{}}
@elextr
Copy link
Member

elextr commented May 23, 2016

The current operation does not auto-close brackets if there is already a matching close bracket.

If a pull request was provided that made it an option for the close bracket to always be added it would probably be accepted.

@ghost
Copy link
Author

ghost commented May 25, 2016

What is a pull and how and by whom could a pull request be made? Is there a way to manipulate the configuration file or something like that to make it possible to add this feature individually? I think it will be really helpful.

@elextr
Copy link
Member

elextr commented May 25, 2016

To be added to Geany itself it has to be coded in C, it is not configuration.

@ghost
Copy link
Author

ghost commented May 25, 2016

I C.
:-)

@b4n
Copy link
Member

b4n commented Jun 9, 2016

IMO the whoel auto-close feature in Geany is broken and should rather be removed and the feature left to a plugin (like auto-close, that seem to do fairly well)

@ghost
Copy link

ghost commented Nov 16, 2018

I have to resort to the plugin too...

@elextr
Copy link
Member

elextr commented Nov 16, 2018

Agree with @b4n, should be removed from core, bet it won't be though. :)

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