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

Run editor from viewer #2156

Open
mc-butler opened this issue Apr 27, 2010 · 12 comments
Open

Run editor from viewer #2156

mc-butler opened this issue Apr 27, 2010 · 12 comments
Labels
area: mcview mcview, the built-in text editor prio: low Minor problem or easily worked around

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2156
Reporter nickk9 (dev.nul@….ru)
Mentions zaytsev (@zyv)

From #83
It would be great if viewer would have key shortcut to run editor on the same file (so i view some file, understand that i need to edit it, then press, say Alt-F4 and get editor on this file from the same position).
I thought that default editor behaviour (placing current line in the middle of the window) not convenient when switching from viewer to editor (thus is results in shifting text), so i modified abovementioned patch to disable such scrolling in this very case.

Note

Original attachments:

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 27, 2010 at 16:21 UTC

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 27, 2010 at 16:22 UTC (comment 1)

  • Description edited
  • Summary changed from Run editor form viewer to Run editor from viewer

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 27, 2010 at 17:51 UTC (comment 2)

Alt+F4 is very dangerous as a shortcut. It is normally stolen by X if you are running mc from within the terminal emulator and does nothing else but closes the current window. Shift+F4 is not too bad, BUT on the other hand, we've got one free place in there (F6), so maybe one could use this instead.

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 27, 2010 at 17:52 UTC (comment 3)

  • Cc set to zaytsev

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on May 8, 2010 at 22:35 UTC (comment 4)

  • Severity changed from no branch to on review
  • Owner set to angel_il
  • Status changed from new to accepted

branch: 2156_run_editor_from_viewer
changest: [fd6c871bcee6f74f480da6ed417916efdc3733d6]

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on May 8, 2010 at 22:46 UTC (comment 5)

  • Version changed from 4.7.1 to master
  • Milestone changed from 4.7 to 4.7.3

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Jul 5, 2010 at 17:35 UTC (comment 6)

  • Milestone changed from 4.7.3 to 4.7.4

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Sep 14, 2010 at 13:19 UTC (comment 7)

  • Milestone changed from 4.7.4 to 4.7.5

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 18, 2010 at 12:58 UTC (comment 8)

  • Milestone changed from 4.7.5 to 4.7
  • Severity changed from on review to on rework

Current 2156_run_editor_from_viewer branch doesn't respect the multi-screen engine. Rework is needed.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Oct 31, 2011 at 11:02 UTC (comment 9)

  • Milestone changed from 4.7 to Future Releases
  • Branch state set to on rework

@mc-butler
Copy link
Author

Changed by ossi (@ossilator) on Jan 5, 2014 at 14:50 UTC (comment 10)

i would move the hex mode to F6 (it's not such a commonly used feature that many people would be very upset about this change) and map the function to F4 to be consistent with mc itself.
the hex mode itself already has a view/edit switch on F2 which should be moved to F4, and save on F6, which should be moved to F2, obviously.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Nov 29, 2016 at 7:16 UTC (comment 11)

  • Status changed from accepted to assigned
  • Owner angel_il deleted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: mcview mcview, the built-in text editor prio: low Minor problem or easily worked around
Development

No branches or pull requests

1 participant