>
 >
 >
 >Have you tried "Learn keys" for that?  If yes, why didn't you like this
 >approach?  I think that the "Learn keys" dialog has two problems - one is
 >that users cannot find it when they need it.  The other is that they don't
 >realize that it can help them.
 >
Yes, it's oriented around the individual keys rather
than the functions associated with them, and it's not
necessarily obvious that you can re-assign the functions
this way.  I messed with it once, and didn't figure it
could help me tweak keystroke combos like that.
Maybe that's a dialog that needs a bunch of work.

 >
 >The other reason is that the selection for directories is in fact
 >supported.  Just add slash at the end of the pattern.  Usability of this
 >feature was obviously neglected, so that not only most users are unaware
 >of it, but even the developers trying to improve this code!
 >
Yes, it's true.  I had no idea.  That's great though.
Y'see how important your experience is?  It was the
directory thing that was bothering me, and there is
already a (little known) feature that does what I want.
I thought there must be some reason someone hadn't
already done this. What about a small patch to add that
info to the select/unselect dialogs:
"Append '/' for directories".  (patch attached :)
As you say, there's many users who don't know that.

 >>Also, the unselect-all SHOULD do directories, shouldn't it.  That's what
 >>it's for, to unselect everything, isn't it?  Right now it won't touch
 >>selected directories.  Isn't that wrong? If it's not wrong, could
 >>someone please explain why?
 >>
 >
 >Maybe because something (just like like you) didn't like writing new
 >dialogs?
 >
Heh heh. I like to check in here first to see if
its worth the time and effort, or I'm just missing
something (usually the case:).


Cheers.


--- mc-cvs-4.6.0-pre1-021003/mc/src/cmd.c       Sun Sep 29 12:26:15 2002
+++ mc-draft-4.6.0-pre1-021003/src/cmd.c        Wed Oct  9 14:01:15 2002
@@ -506,7 +508,7 @@
     int c;
     int dirflag = 0;
 
-    reg_exp = input_dialog (_(" Select "), "", easy_patterns ? "*" : ".");
+    reg_exp = input_dialog (_(" Select "), "Append \"/\" for directories.", 
+easy_patterns ? "*" : ".");
     if (!reg_exp)
        return;
     
@@ -558,7 +560,7 @@
     int c;
     int dirflag = 0;
 
-    reg_exp = input_dialog (_(" Unselect "),"", easy_patterns ? "*" : ".");
+    reg_exp = input_dialog (_(" Unselect "),"Append \"/\" for directories.", 
+easy_patterns ? "*" : ".");
     if (!reg_exp)
        return;
     

Reply via email to