[hackers] [dmenu] updated manpage, changed keybinds || Connor Lane Smith

From: <hg_AT_suckless.org>
Date: Sun, 20 Jun 2010 14:04:21 +0000 (UTC)

changeset: 312:2fe12779814a
tag: tip
user: Connor Lane Smith <cls_AT_lubutu.com>
date: Sun Jun 20 15:04:15 2010 +0100
files: dmenu.1 dmenu.c
description:
updated manpage, changed keybinds
M- binds tend to be wm level, and there were up to 3 binds for the same action
M-{hjkl} also no longer made sense in vlist mode

diff -r daefdd7ce876 -r 2fe12779814a dmenu.1
--- a/dmenu.1 Sun Jun 20 01:19:17 2010 +0100
+++ b/dmenu.1 Sun Jun 20 15:04:15 2010 +0100
@@ -14,12 +14,22 @@
 .RB [ \-sb " <color>]"
 .RB [ \-sf " <color>]"
 .RB [ \-v ]
+
+.B dmenu_run
+[<options...>]
+
+.BR dmenu_path
 .SH DESCRIPTION
 .SS Overview
 dmenu is a generic menu for X, originally designed for
 .BR dwm (1).
 It manages huge amounts (up to 10.000 and more) of user defined menu items
 efficiently.
+
+dmenu_run is a dmenu script used by dwm which lists executables in the user's PATH
+and executes the selected item.
+
+dmenu_path is a script used by dmenu_run to find and cache a list of executables.
 .SS Options
 .TP
 .B \-i
@@ -33,7 +43,7 @@
 .TP
 .B \-l <lines>
 activates vertical list mode.
-The given number of lines will be displayed. Window height will get adjusted.
+The given number of lines will be displayed. Window height will be adjusted.
 .TP
 .B \-fn <font>
 defines the font.
@@ -60,20 +70,9 @@
 menu. When the user selects an item or enters any text and presses Return, his/her
 choice is printed to standard output and dmenu terminates.
 .P
-dmenu is completely controlled by the keyboard. The following keys are recognized:
-.TP
-.B Any printable character
-Appends the character to the text in the input field. This works as a filter:
-only items containing this text will be displayed.
-.TP
-.B Left/Right (Up/Down) (Mod1\-h/Mod1\-l)
-Select the previous/next item.
-.TP
-.B PageUp/PageDown (Mod1\-k/Mod1\-j)
-Select the first item of the previous/next 'page' of items.
-.TP
-.B Home/End (Mod1\-g/Mod1\-G)
-Select the first/last item.
+dmenu is completely controlled by the keyboard. Besides standard Unix line editing,
+and item selection (Up/Down or Left/Right, PageUp/PageDown, Home/End), the following
+keys are recognized:
 .TP
 .B Tab (Control\-i)
 Copy the selected item to the input field.
@@ -84,24 +83,19 @@
 on termination.
 .TP
 .B Shift\-Return (Control\-Shift\-j)
-Confirm selection and quit (print the text in the input field to standard output).
+Confirm input and quit (print the text in the input field to standard output).
 Returns
 .B 0
 on termination.
 .TP
-.B Escape (Control\-bracketleft)
+.B Escape (Control\-c)
 Quit without selecting an item. Returns
 .B 1
 on termination.
 .TP
-.B Backspace (Control\-h)
-Remove a character from the input field.
-.TP
-.B Control\-u
-Remove all characters from the input field.
-.TP
-.B Control\-w
-Remove all characters of current word from the input field.
+.B Control\-y
+Pastes the X selection into the input field. This requires
+.BR sselp (1).
 .SH SEE ALSO
 .BR dwm (1),
-.BR wmii (1) .
+.BR wmii (1).
diff -r daefdd7ce876 -r 2fe12779814a dmenu.c
--- a/dmenu.c Sun Jun 20 01:19:17 2010 +0100
+++ b/dmenu.c Sun Jun 20 15:04:15 2010 +0100
@@ -355,17 +355,23 @@
         /* first check if a control mask is omitted */
         if(e->state & ControlMask) {
                 switch(tolower(ksym)) {
- default: /* ignore other control sequences */
+ default:
                         return;
                 case XK_a:
                         ksym = XK_Home;
                         break;
+ case XK_b:
+ ksym = XK_Left;
+ break;
                 case XK_c:
                         ksym = XK_Escape;
                         break;
                 case XK_e:
                         ksym = XK_End;
                         break;
+ case XK_f:
+ ksym = XK_Right;
+ break;
                 case XK_h:
                         ksym = XK_BackSpace;
                         break;
@@ -378,6 +384,12 @@
                 case XK_k:
                         text[cursor] = '\0';
                         break;
+ case XK_n:
+ ksym = XK_Down;
+ break;
+ case XK_p:
+ ksym = XK_Up;
+ break;
                 case XK_u:
                         memmove(text, text + cursor, sizeof text - cursor + 1);
                         cursor = 0;
@@ -393,31 +405,7 @@
                                 match(text);
                         }
                         break;
- }
- }
- if(CLEANMASK(e->state) & Mod1Mask) {
- switch(ksym) {
- default:
- return;
- case XK_h:
- ksym = XK_Left;
- break;
- case XK_l:
- ksym = XK_Right;
- break;
- case XK_j:
- ksym = XK_Next;
- break;
- case XK_k:
- ksym = XK_Prior;
- break;
- case XK_g:
- ksym = XK_Home;
- break;
- case XK_G:
- ksym = XK_End;
- break;
- case XK_p:
+ case XK_y:
                         {
                                 FILE *fp;
                                 char *s;
@@ -453,6 +441,8 @@
                 match(text);
                 break;
         case XK_Delete:
+ if(cursor == len)
+ return;
                 for(i = 1; cursor + i < len && !IS_UTF8_1ST_CHAR(text[cursor + i]); i++);
                 memmove(text + cursor, text + cursor + i, sizeof text - cursor);
                 match(text);
@@ -482,18 +472,20 @@
                 calcoffsets();
                 break;
         case XK_Left:
+ if(cursor > 0 && (!sel || !sel->left)) {
+ while(cursor-- > 0 && !IS_UTF8_1ST_CHAR(text[cursor]));
+ break;
+ }
+ if(lines > 0)
+ return;
         case XK_Up:
- if(sel && sel->left){
- sel = sel->left;
- if(sel->right == curr) {
- curr = prev;
- calcoffsets();
- }
+ if(!sel || !sel->left)
+ return;
+ sel = sel->left;
+ if(sel->right == curr) {
+ curr = prev;
+ calcoffsets();
                 }
- else if(cursor > 0)
- while(cursor-- > 0 && !IS_UTF8_1ST_CHAR(text[cursor]));
- else
- return;
                 break;
         case XK_Next:
                 if(!next)
@@ -516,18 +508,20 @@
                 running = False;
                 return;
         case XK_Right:
+ if(cursor < len) {
+ while(cursor++ < len && !IS_UTF8_1ST_CHAR(text[cursor]));
+ break;
+ }
+ if(lines > 0)
+ return;
         case XK_Down:
- if(cursor < len)
- while(cursor++ < len && !IS_UTF8_1ST_CHAR(text[cursor]));
- else if(sel && sel->right) {
- sel = sel->right;
- if(sel == next) {
- curr = next;
- calcoffsets();
- }
+ if(!sel || !sel->right)
+ return;
+ sel = sel->right;
+ if(sel == next) {
+ curr = next;
+ calcoffsets();
                 }
- else
- return;
                 break;
         case XK_Tab:
                 if(!sel)
Received on Sun Jun 20 2010 - 14:04:21 UTC

This archive was generated by hypermail 2.2.0 : Sun Jun 20 2010 - 14:12:04 UTC