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

Logout in oblogout ask confimation #275

Closed
holmeslinux opened this Issue Jan 27, 2016 · 7 comments

Comments

Projects
None yet
4 participants
@holmeslinux

hi all,
i'm having doubts about the logout command in oblogout.

=> menu have: <Exit label="Logout" confirm="false"/> and does not ask confirmation.
=> oblogout have: jwm -exit and ask for confirmation.

image here https://forum.manjaro.org/index.php?topic=30486.msg252430#msg252430, ask for conformation!

i would like to know how to avoid confirmation.

vlw fwi, Holmes

@holmeslinux

This comment has been minimized.

Show comment
Hide comment
@holmeslinux

holmeslinux Jan 28, 2016

hi guys, i found an option that works, but i do not know if it's correct. here works, see

pkill -u ${USER}

it is correct?

vlw fwi, Holmes

hi guys, i found an option that works, but i do not know if it's correct. here works, see

pkill -u ${USER}

it is correct?

vlw fwi, Holmes

@rdnvndr

This comment has been minimized.

Show comment
Hide comment
@rdnvndr

rdnvndr Jan 28, 2016

Contributor

It works :

=> menu have: and does not ask confirmation.
=> oblogout have: jwm -exit and ask for confirmation.>

Please write a version of JWM.

Contributor

rdnvndr commented Jan 28, 2016

It works :

=> menu have: and does not ask confirmation.
=> oblogout have: jwm -exit and ask for confirmation.>

Please write a version of JWM.

@Israel-D

This comment has been minimized.

Show comment
Hide comment
@Israel-D

Israel-D Jan 28, 2016

Contributor

I set this in a non used menu JUST for this purpose

<RootMenu height="0" onroot="z">
        <Exit label="Logout" confirm="false" icon="system-logout.svg"/>
    </RootMenu>

This works for me.
It has worked since Joe implemented a-z menus
I didn't use it before, so I cannot tell if it worked before.
pkill -u $USER is not a really nice way to do things IMO, but it is your distro :)
If you guys want an example of a working fully functional jwmrc
You can check the ToriOS version
http://bazaar.launchpad.net/~torios-dev/torios/trunk/view/head:/etc/skel/.jwmrc
This has a full set of key bindings that are compliant with most GNU/OSX/Windows key bindings.
We also have a bunch of scripts to do a lot of the work for our users, such as icon theme generation, volume control (using amixer) we have scripts to generate menus, but we also have binary programs for the applications, and 'places' menus. It might be good to collaborate at some point if you are interested.

Contributor

Israel-D commented Jan 28, 2016

I set this in a non used menu JUST for this purpose

<RootMenu height="0" onroot="z">
        <Exit label="Logout" confirm="false" icon="system-logout.svg"/>
    </RootMenu>

This works for me.
It has worked since Joe implemented a-z menus
I didn't use it before, so I cannot tell if it worked before.
pkill -u $USER is not a really nice way to do things IMO, but it is your distro :)
If you guys want an example of a working fully functional jwmrc
You can check the ToriOS version
http://bazaar.launchpad.net/~torios-dev/torios/trunk/view/head:/etc/skel/.jwmrc
This has a full set of key bindings that are compliant with most GNU/OSX/Windows key bindings.
We also have a bunch of scripts to do a lot of the work for our users, such as icon theme generation, volume control (using amixer) we have scripts to generate menus, but we also have binary programs for the applications, and 'places' menus. It might be good to collaborate at some point if you are interested.

@holmeslinux

This comment has been minimized.

Show comment
Hide comment
@holmeslinux

holmeslinux Jan 29, 2016

hi guys, my idea is to add the jwm -exit in oblogout. however, I would like to remove the confirmation of JWM:

Exit JWM
Are you sure?
OK Cancel

how to remove the Exit in oblogout?

my oblogout:

[settings]
usehal = false

[looks]
opacity = 40
bgcolor = black
buttontheme = GrayOrange
buttons = cancel, lock, suspend, logout, restart, shutdown

[shortcuts]
cancel = Escape
lock = K
suspend = U
logout = L
restart = R
shutdown = S

[commands]
lock = i3lock -i $HOME/.i3lock/fly-chauvin.png
suspend = systemctl suspend
logout = jwm -exit
restart = systemctl reboot
shutdown = systemctl poweroff

ps: version JWM = 2.3.4.

vlw fwi, Holmes

hi guys, my idea is to add the jwm -exit in oblogout. however, I would like to remove the confirmation of JWM:

Exit JWM
Are you sure?
OK Cancel

how to remove the Exit in oblogout?

my oblogout:

[settings]
usehal = false

[looks]
opacity = 40
bgcolor = black
buttontheme = GrayOrange
buttons = cancel, lock, suspend, logout, restart, shutdown

[shortcuts]
cancel = Escape
lock = K
suspend = U
logout = L
restart = R
shutdown = S

[commands]
lock = i3lock -i $HOME/.i3lock/fly-chauvin.png
suspend = systemctl suspend
logout = jwm -exit
restart = systemctl reboot
shutdown = systemctl poweroff

ps: version JWM = 2.3.4.

vlw fwi, Holmes

@joewing

This comment has been minimized.

Show comment
Hide comment
@joewing

joewing Jan 29, 2016

Owner

The way JWM currently works is somewhat broken. The exit confirmation is a global setting even though it appears that it can be configured on a per-item basis in the configuration file. To fix this, JWM should treat it as configuration for each instance of "Exit" (or "Restart") in the configuration. Also, JWM should never show an exit confirmation when "-exit" is invoked from the command line.

Owner

joewing commented Jan 29, 2016

The way JWM currently works is somewhat broken. The exit confirmation is a global setting even though it appears that it can be configured on a per-item basis in the configuration file. To fix this, JWM should treat it as configuration for each instance of "Exit" (or "Restart") in the configuration. Also, JWM should never show an exit confirmation when "-exit" is invoked from the command line.

@joewing

This comment has been minimized.

Show comment
Hide comment
@joewing

joewing Jan 29, 2016

Owner

This should be fixed in snapshot 1370.

Owner

joewing commented Jan 29, 2016

This should be fixed in snapshot 1370.

@holmeslinux

This comment has been minimized.

Show comment
Hide comment
@holmeslinux

holmeslinux Jan 29, 2016

thank you Joe and i hope it can be resolved in the next version!

vlw fwi, Holmes :)

thank you Joe and i hope it can be resolved in the next version!

vlw fwi, Holmes :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment