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

Midnight Commander 4.8.19 Keyboard Regression #3807

Closed
mc-butler opened this issue Apr 7, 2017 · 5 comments
Closed

Midnight Commander 4.8.19 Keyboard Regression #3807

mc-butler opened this issue Apr 7, 2017 · 5 comments
Labels
area: tty Interaction with the terminal, screen libraries prio: high Serious problem that could block progress

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3807
Reporter Axanar (aliakc@….de)

Hello

I would like to inform, that there is a keybinding regression inside mc-4.8.19, that I reported to Fedora today (They put a new release in the repository).

https://bugzilla.redhat.com/show_bug.cgi?id=1440110

Let me explain:

I've been using mc since it came out (with other words: many many years). It's one of my main tools that I regulary and intensively use. And as you may guess, most keyboard shortcuts are memorized (at least the ones I use). Till mc-4.8.18 everything worked but with mc-4.8.19 keybindings have changed...

Example:

When mc starts, the usual keybindings are (sorry if I name them wrong)

F5 = Copy
F6 = Move
Shift+F5 = Copy As
Shift+F6 = Move As

With 4.8.19 they have changed.

F5 = Copy
F6 = Move
Shift+F3 = Copy As
Shift+F4 = Move As
Shift+F6 = Delete (if you have confirm delete disabled then everything will be wiped)

Previous Shift+F3 and Shift+F4 functionality are overwritten (usually Shift+F3 opens internal editor).

I strongly believe, that this is a regression and like to have it fixed to the behaviour it had before.

I also consider this quite critical because of the expected behaviour that Shift+F6 usually have a popup (dialog) coming up, where you can change paths or filenames...

With the current behaviour you could accidently remove all your files and directories because Shift+F6 reflects to "delete" now, if you have the said confirmation flag inside the preferences disabled (delete without feedback).

Running Fedora, Xorg, XFCE-4.12

Downgrading to mc-4.8.18 solves the issues. So it must be a regression somewhere the transition to mc-4.8.19.

Btw: This issue also exists (for years) if you switch to Console. Copy and Move are incorrectly placed to F3 and F4 rather than F5 and F6 (as even the Keybar tells). So what affected Console for years now also affects mc when running under Xorg within a terminal.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 7, 2017 at 17:05 UTC

Replying to Axanar (#3807):

When mc starts, the usual keybindings are (sorry if I name them wrong)

F5 = Copy
F6 = Move
Shift+F5 = Copy As
Shift+F6 = Move As

With 4.8.19 they have changed.

This is impossible. There no any such changes made in mc.

F5 = Copy
F6 = Move
Shift+F3 = Copy As
Shift+F4 = Move As
Shift+F6 = Delete (if you have confirm delete disabled then everything will be wiped)

Everything used fine for me for years.

Downgrading to mc-4.8.18 solves the issues.

This is very strange.

So it must be a regression somewhere the transition to mc-4.8.19.

I can't agree.

Btw: This issue also exists (for years) if you switch to Console. Copy and Move are incorrectly placed to F3 and F4 rather than F5 and F6 (as even the Keybar tells). So what affected Console for years now also affects mc when running under Xorg within a terminal.

#3254

@mc-butler
Copy link
Author

Changed by Axanar (aliakc@….de) on Apr 7, 2017 at 18:19 UTC (comment 2)

Ok I did the following:

I've set up an development environment on my system. Everything needed to build mc on Fedora.

dnf builddep mc
dnf install gcc make autoconf automake libtool gettext slang-devel

Then I downloaded mc-4.8.18.tar.bz2, ran configure --prefix=/usr, make, make install.

Works as expected!

Then I downloaded mc-4.8.19.tar.bz2, ran the same arguments as above.. but removed 4.8.18 before.

Works as expected! No issues, as I initially reported...

But!

http://pkgs.fedoraproject.org/cgit/rpms/mc.git/commit/?id=589373c36dbfe5fddd1e5a187cc60b7580c9ade9

The Fedora devs seem to have changed from using slang to using ncurses now...

So I just recompiled 4.8.18 again (the usual stuff, removing old installation and then the above parameters again).

Fails! With NCurses the issues as described by me shows up. I don't think I need to recompile 4.8.19 with NCurses then.

So this is a NCurses or underlaying specific issue as described in the bugreport.

#3254

Now.... How to solve this issue ? And what would be the proper answer that I can give within the Fedora bugreport so the Fedora developers understand the issue... It's hard using mc with misaligned Fx keys...

@mc-butler
Copy link
Author

Changed by kloczek (kloczko.tomasz@….com) on Apr 7, 2017 at 21:16 UTC (comment 3)

Now.... How to solve this issue ? And what would be the proper answer that I can give within the Fedora bugreport so the Fedora developers understand the issue... It's hard using mc with misaligned Fx keys...

Seems like ncurses misc/terminfo.src terminals definitions needs to be fixed.
It may more such reports and all of them should have included reporting $TERM variable value.

PS. I was responsible for switching mc in Fedora from SLang to ncurses.
I'm using quite long time mc with ncurses on Solaris. However I must confess that I never been using ctrl and shift key combinations :P

@mc-butler
Copy link
Author

Changed by zaytsev (@zyv) on Apr 7, 2017 at 21:40 UTC (comment 4)

@kloczek, since it seems that you're doing some work on mc SPEC in Fedora now, could you please also have a look at

https://bugzilla.redhat.com/show_bug.cgi?id=1383054

I've asked several times to fix this already, but no dice so far. Thanks!

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Apr 8, 2017 at 6:03 UTC (comment 5)

  • Component changed from mc-core to mc-tty
  • Status changed from new to closed
  • Milestone Future Releases deleted
  • Resolution set to duplicate

Closed as duplicate of #3254.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: tty Interaction with the terminal, screen libraries prio: high Serious problem that could block progress
Development

No branches or pull requests

1 participant