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

(mandriva) add ctrl-l as hotkey for quick-chdir #195

Closed
mc-butler opened this issue Jan 16, 2009 · 13 comments
Closed

(mandriva) add ctrl-l as hotkey for quick-chdir #195

mc-butler opened this issue Jan 16, 2009 · 13 comments
Assignees
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress ver: 4.6.1 Reproducible in version 4.6.1
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/195
Reporter metux (@metux)
Keywords mandriva

http://svn.mandriva.com/cgi-bin/viewvc.cgi/packages/cooker/mc/current/SOURCES/mc-4.6.2-shortcut.patch?view=markup

@mc-butler
Copy link
Author

Changed by metux (@metux) on Jan 16, 2009 at 3:37 UTC (comment 1)

  • Type changed from defect to enhancement

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 18, 2009 at 9:57 UTC (comment 2)

It's a bad idea. Currentrly Ctrl-l hotkey is used to refresh the screen (see src/screen.c(dialog_handle_key)).

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Jan 18, 2009 at 11:03 UTC (comment 3)

  • Status changed from new to accepted
  • Owner set to slavazanko

It's a bad idea.

+1. In editor CTRL+L used too.

refresh the screen

'Repaint' more exactly. :)

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Jan 18, 2009 at 11:03 UTC (comment 4)

  • Resolution set to invalid
  • Status changed from accepted to testing

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Jan 18, 2009 at 11:04 UTC (comment 5)

  • Keywords changed from review, mandriva to mandriva

@mc-butler
Copy link
Author

Changed by winnie (@winnieXY) on Jan 18, 2009 at 21:22 UTC (comment 6)

  • Status changed from testing to closed

@mc-butler
Copy link
Author

Changed by Adam Williamson on Jan 30, 2009 at 11:01 UTC

  • Resolution invalid deleted
  • Status changed from closed to reopened

On Sun, 2009-01-18 at 21:22 +0000, Ticket System wrote:

#195: (mandriva) add ctrl-l as hotkey for quick-chdir


Reporter: metux | Owner: slavazanko

Type: enhancement | Status: closed

Priority: major | Milestone: 4.7

Component: mc-core | Version: 4.6.1

Resolution: invalid | Keywords: mandriva

Blocking: | Blockedby:


Changes (by winnie):

  • status: testing => closed

On this one - I don't think closing it because the shortcut I happened
to choose is a bad one is a valid resolution. The *reason* this patch
was introduced is still valid. See the original Mandriva bug:

http://qa.mandriva.com/show_bug.cgi?id=44201

the problem is that the ctrl-\ combination is unavailable on some
keyboard layouts. This problem is still a valid one. It would be better
to re-open the bug and pick some other available combination, rather
than just not address the problem.

See the discussion there on trying to find an available combo, also.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Jan 30, 2009 at 11:29 UTC (comment 7.8)

Replying to Adam Williamson:

pick some other available combination, rather than just not address the problem.

Maybe Alt-/

@mc-butler
Copy link
Author

Changed by styx (@styx) on Jan 31, 2009 at 11:00 UTC (comment 9)

I think we shouldn't to guess now which combo to use. At first we need to release #212 and then create some predefined schemes for tree view and editor. Of course we will have legacy bindings and for editor something like: emacs/vim/etc.

@mc-butler
Copy link
Author

Changed by Enrico Weigelt on Feb 9, 2009 at 6:41 UTC

Perhaps we should think about configurable shortcuts in general ?
(maybe for 4.7 ?)

@mc-butler
Copy link
Author

Changed by winnie (@winnieXY) on Feb 9, 2009 at 7:24 UTC (comment 11)

  • Blocked by set to #212

Please have a look on #212. This is about configurable shortcuts in general.

@mc-butler
Copy link
Author

Changed by styx (@styx) on Feb 23, 2009 at 12:18 UTC (comment 12)

  • Status changed from reopened to closed
  • Resolution set to wontfix

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Sep 3, 2009 at 10:57 UTC

  • Blocked by #212 deleted

@mc-butler mc-butler closed this as not planned Won't fix, can't repro, duplicate, stale Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress ver: 4.6.1 Reproducible in version 4.6.1
Development

No branches or pull requests

2 participants