-
-
Notifications
You must be signed in to change notification settings - Fork 78
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
Update to config #137
Update to config #137
Conversation
lgsobalvarro
commented
Jun 24, 2020
- IconMan now uses colorset 10 instead 0, to honor Wiki conventions (Thanks to somiaj)
- dmenu_run uses -nb '#2b4e5e' so it'll integrate better with the new colorset (Suggestion by Master_P_the_Gu)
- Edited menu option xemacs for emacs since xemacs is in maintenance mode and emacs seems to be more popular this days.
pull from fvwmorg
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @lgsobalvarro,
Thanks for this! Aside from my comments herein, everything else looks good.
default-config/config
Outdated
@@ -285,7 +285,7 @@ AddToMenu MenuFvwmRoot "Fvwm" Title | |||
+ "&Programs%icons/programs.png%" Popup MenuPrograms | |||
+ "XDG &Menu%icons/apps.png%" Popup XDGMenu | |||
+ "&XTerm%icons/terminal.png%" Exec exec $[infostore.terminal] | |||
Test (x dmenu_run) + "Run command" Exec exec dmenu_run | |||
Test (x dmenu_run) + "Run command" Exec exec dmenu_run -nb "#2b4e5e" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should move this colour definition to using infostore, and use it here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a good suggestion, it would simplify things. Do you suggest only changing the color definition or use infostore for the full command? Perhaps the full command option could be more convenient for those who want to switch to Rofi or something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. Even better. Go for it!
default-config/config
Outdated
@@ -462,7 +462,7 @@ Silent Key F2 A C GotoDesk 0 1 | |||
Silent Key F3 A C GotoDesk 0 2 | |||
Silent Key F4 A C GotoDesk 0 3 | |||
Silent Key Super_R A A Exec exec $[infostore.terminal] | |||
Silent Key Space A M Exec exec dmenu_run | |||
Silent Key Space A M Exec exec dmenu_run -nb "#2b4e5e" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here.