Ticket #1946 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Cursor beyond EOL has unusual default

Reported by: tillea Owned by: angel_il
Priority: major Milestone: 4.7.1
Component: mcedit Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset: committed-master committed-stable

Description

Hi,
with the current default setting of cursor_beyond_eol mcedit is behaving different than any other editor I know which IMHO at least a badly choosen default behaviour. I would strongly vote to keep the original behaviour and leave this new (??) behaviour as a configuration option especially because it is not really obvious to find this option.
Kind regards

Andreas.

Change History

comment:1 Changed 6 years ago by andrew_b

  • Version changed from version not selected to 4.7.0
  • Component changed from mc-core to mcedit

comment:2 follow-up: ↓ 6 Changed 6 years ago by ossi

do you mean ticket 1801 by any chance?

comment:3 Changed 6 years ago by angel_il

I would strongly vote to keep the original behaviour...

by default you have original behaviour, this option switced off by default, just do not turned on.

comment:4 follow-up: ↓ 7 Changed 6 years ago by angel_il

and... what you mean when you say "other editor"?

comment:5 follow-up: ↓ 9 Changed 6 years ago by angel_il

and... all new features switched off by default.

comment:6 in reply to: ↑ 2 Changed 6 years ago by tillea

Replying to ossi:

do you mean ticket 1801 by any chance?

Uhm yes. Sorry for bad research in advance

Andreas.

comment:7 in reply to: ↑ 4 ; follow-up: ↓ 11 Changed 6 years ago by tillea

Replying to angel_il:

and... what you mean when you say "other editor"?

Any editor I ever have used in > 25 years. I never have seen an editor where I can "navigate in the void" and I do not have the slightest idea for what this is useful.

Andreas.

comment:8 Changed 6 years ago by ossi

hmmm, no, we're not talking about the same thing then. the "navigation in the void" is (or at least was) a pretty common feature in programmers' editors, like the good ol' borland dos IDEs. i only object to the fact that it does not behave like in these ...

comment:9 in reply to: ↑ 5 ; follow-up: ↓ 10 Changed 6 years ago by andrew_b

Replying to angel_il:

and... all new features switched off by default.

Really? :) This feature in ON.

edit/edit.c:78

int option_cursor_beyond_eol = 1;

comment:10 in reply to: ↑ 9 Changed 6 years ago by angel_il

Replying to andrew_b:

Replying to angel_il:

and... all new features switched off by default.

Really? :) This feature in ON.

edit/edit.c:78

int option_cursor_beyond_eol = 1;

ok, need fix it :)

comment:11 in reply to: ↑ 7 ; follow-up: ↓ 12 Changed 6 years ago by angel_il

Replying to tillea:

Replying to angel_il:

and... what you mean when you say "other editor"?

Any editor I ever have used in > 25 years.

ok, borland editors, far editor, emacs, lazarus, notepad++, codeblock, vim... and many many editors...

I never have seen an editor where I can "navigate in the void" and I do not have the slightest idea for what this is useful.

just try select vertical block with/without this option... feel different...

...and this options will be switched of by default...

comment:12 in reply to: ↑ 11 Changed 6 years ago by tillea

Replying to angel_il:

ok, borland editors, far editor, emacs, lazarus, notepad++, codeblock, vim... and many many editors...

It might be that all these editors might have an option to change their behaviour but if I'm at the end of a line and press the <right-arrow> key I end up in the next line or the cursor just stops there and not in an area where the edited file has no characters at all.

just try select vertical block with/without this option... feel different...

Working with vertical blocks is a specific modus. Works perfect for instance with Emacs if you insert some spaces in the first / last line if needed - I do not remember how all the other editors behave in this modus.

...and this options will be switched of by default...

That's great and was the issue of my bug report.
Thanks

Andreas.

comment:13 Changed 6 years ago by angel_il

  • Owner set to angel_il
  • Status changed from new to accepted
  • Version changed from 4.7.0 to master
  • Milestone changed from 4.7 to 4.7.1

comment:14 Changed 6 years ago by angel_il

  • severity changed from no branch to on review

branch: 1946_cursor_beyond_EOL_fix
changeset: 8d62484b725416686de5a410b390acc668189c27

comment:15 Changed 6 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:16 Changed 6 years ago by slavazanko

  • Votes for changeset changed from andrew_b to andrew_b slavazanko
  • severity changed from on review to approved

comment:17 Changed 6 years ago by angel_il

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b slavazanko to committed-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:18 Changed 6 years ago by angel_il

  • Status changed from testing to closed

comment:19 Changed 6 years ago by slavazanko

  • Votes for changeset changed from committed-master to committed-master commited-stable
Note: See TracTickets for help on using tickets.