Skip to content
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

Add reset themes #58

Open
wants to merge 65 commits into
base: master
Choose a base branch
from
Open

Add reset themes #58

wants to merge 65 commits into from

Conversation

joslinpr
Copy link

@joslinpr joslinpr commented Apr 4, 2024

The themes use set -goq. From the man page:

  If -g is given, the global session or window option is set.
 ...
  The -o flag prevents setting an option that is already set and the -q flag suppresses errors about unknown or ambiguous options.

As a result, once a theme is set, you cannot change it. This PR creates two themes for developers. The "reset.tmuxtheme" uses set -gq, so that it overrides existing variables.
The "reset-debug.tmuxtheme" uses set -g, so that variables are overridden, and error messages are not suppressed.

Steps to demonstrate the problem:
Load a theme, then load a second theme. The values of the first theme remain.

An alternate enhancement would be to use set -gq in all themes. I do not know why goq was chosen, so I have not changed those themes.

Paul R. Joslin (Work) and others added 30 commits April 4, 2024 09:07
Reorder Includes so overrides work
Paul: Use only 2 left and right status slots
Use local apply and affix files
Make after changes
Make _apply and _themepack-options part of Paul theme
Make Paul/block actually blocky.
Rebuild all themes
Paul R. Joslin (Work) and others added 30 commits April 23, 2024 13:32
All: Remove -gq, use -g
All: Use -gwF where needed
block-simple: Fix separator
Add settitle, to change session, window, pane titles
Update pane-border-format
Override pane-border-format for block-simple
Add SetEnv to setup dev environment
Tools: Use $HOME instead of hard coded directory
PANE - In 2.7, turn pane-status on, set-window-options instead of
set-option
Makefile cleanup
Minify remove most comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant