Repository navigation
Add DISABLE_WINDOW_BUTTONS to remove all titlebar buttons - #194
Closed
junkerderprovinz wants to merge 1 commit into
Closed
junkerderprovinz wants to merge 1 commit into
junkerderprovinz wants to merge 1 commit into
Conversation
Member
|
Can you pr this against dev? In general dev stuff will all get ported over to all the flavors in bulk when we are ready for a waterfall. |
junkerderprovinz
force-pushed
the
feat/disable-window-buttons
branch
from
October 2, 2026 00:57
a3a01a7 to
1f4cf99
Compare
Member
|
implemented in dev |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #186.
DISABLE_WINDOW_BUTTONS=trueremoves minimize, maximize and close from the titlebar. The two new blocks sit next to the existingDISABLE_CLOSE_BUTTONones: they stripI,MandCfrom the openboxtitleLayout, andiconify,maxandclosefrom the labwc layout. The labwc match works on the tokens rather than on the default string, so it still ends aticon:whenDISABLE_CLOSE_BUTTONhas already takencloseout.This only covers the buttons. The titlebar mousebinds are unchanged, so double-clicking the openbox titlebar still toggles maximize.
I left
HARDEN_OPENBOXalone so existing kiosk setups keep the titlebar they have now. If you'd rather it turned this on too, that's one more export next toDISABLE_CLOSE_BUTTON, and I can add it.Tested on the published
debiantrixieimage with thisrunmounted over the original, under X11 and withPIXELFLUX_WAYLAND=true, each with and withoutHARDEN_OPENBOX:With
HARDEN_OPENBOX, the locked rc.xml under/configends up the same. In the web client the titlebar shows no buttons, while the same image without the variable still draws all three.The
runfile is identical on alpine324, arch, dev, fedora44, kali and ubunturesolute. On the alpine324 image, which uses busybox sed, the same four runs give the same result.Once this lands I'll add the variable to the switch tables on the selkies security.md and building-images.md pages in docker-documentation.