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

feat(status): window specific separator config #198

Merged

Conversation

mklbravo
Copy link
Contributor

@mklbravo mklbravo commented Apr 23, 2024

This pull request:

  • Add window_type to build_window_format
  • If current window separator is configured it will override global configuration
  • Update README.md to reflect new available options

Allowed new configuration options:

# Current type window configuration. If set, it will override the global configuration
set -g @catppuccin_window_current_left_separator "█"
set -g @catppuccin_window_current_middle_separator " 󰿟 "
set -g @catppuccin_window_current_right_separator "█"

Fixes #197

@mklbravo
Copy link
Contributor Author

Screenshot 2024-04-23 at 23 37 01

This theme can be configured like this:

# Window global configuration
set -g @catppuccin_window_left_separator "█"
set -g @catppuccin_window_middle_separator "█ "
set -g @catppuccin_window_number_position "left"
set -g @catppuccin_window_right_separator "█"

# Window current configuration
set -g @catppuccin_window_current_middle_separator " 󰿟 "
set -g @catppuccin_window_current_fill "all"

# Window default configuration
set -g @catppuccin_window_default_fill "number"

Without this PR it will look like this:
Screenshot 2024-04-23 at 23 09 10

@mklbravo mklbravo force-pushed the feat/window-specific-separator-config branch from ea68c51 to dac1a87 Compare April 26, 2024 09:26
@mklbravo mklbravo marked this pull request as ready for review April 26, 2024 09:40
@mklbravo mklbravo force-pushed the feat/window-specific-separator-config branch from 2816e38 to 4589db4 Compare May 2, 2024 09:35
@mklbravo
Copy link
Contributor Author

mklbravo commented May 6, 2024

Thanks for the approval @vvorobev !

README.md Outdated Show resolved Hide resolved
window/window_default_format.sh Outdated Show resolved Hide resolved
window/window_current_format.sh Outdated Show resolved Hide resolved
@mklbravo mklbravo force-pushed the feat/window-specific-separator-config branch from 4589db4 to 91a8276 Compare July 29, 2024 14:28
Copy link
Contributor

@vdbe vdbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update branch merge conflict with main

README.md Outdated Show resolved Hide resolved
@mklbravo mklbravo force-pushed the feat/window-specific-separator-config branch from 91a8276 to b4913d7 Compare September 5, 2024 16:18
@mklbravo mklbravo force-pushed the feat/window-specific-separator-config branch from b4913d7 to 6b42536 Compare September 6, 2024 21:35
@mklbravo mklbravo requested a review from vdbe September 6, 2024 21:39
Copy link
Contributor

@vdbe vdbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really like your config.

@vdbe vdbe changed the title Feat: Window specific separator config feat(status): window specific separator config Sep 7, 2024
@vdbe vdbe merged commit 8276c5a into catppuccin:main Sep 7, 2024
3 checks passed
@mklbravo
Copy link
Contributor Author

mklbravo commented Sep 7, 2024

Thank you @vdbe !

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.

Feature Request: Specific 'separator' configuration for each window type
3 participants