Conversation
| * Selects all text in the given line. | ||
| * @param line The line number to select all text in it. | ||
| */ | ||
| public final void selectLine(final int line) |
There was a problem hiding this comment.
Hey @urbanskimichal! I think I'm in favor of the change if it isn't confusing to new users (I don't think it would be, it's common behavior for a lot of editors). However, could you say more about why you made this public?
|
Hey there @urbanskimichal! I am doing a integration build so I modified the above to private. It's still ok to change back but wanted to make that edit before bringing this branch into integration. That is not reflected in your fork. |
|
See urbanskimichal#1 |
|
This is available on an integration branch at https://github.com/sampottinger/processing4 ( |
|
If we can get a rebased version with that change to |
|
I resolved conflicts, should I do anything else? What does it mean "to private"? |
|
I think @benfry wants selectLine to be made into a private method instead of a public method? (If it’s possible) |
|
All set for beta 4; thanks! |
|
So the change is included? |
|
It will be in beta 4, the next release. |
|
This pull request has been automatically locked. Pull requests that have been closed are automatically locked 30 days after the last comment. |
This is proposal of the feature that I personally miss the most in PDE - copying / cutting whole lines.
When no text is selected and one presses Ctrl+c (Cmd+c) or Ctrl+x (Cmd+x), the whole text in the line on which the caret is, will be coped or cut.
It allows for easy lines duplicating etc.