fix: replace @_ctp_status_bg with @thm_surface_0 in module configurtion to properly show module separation #380

xafarr · 2024-10-17T01:19:31Z

Before the change:
With @catppuccin_status_connect_separator set to yes or no

After the change:
With @catppuccin_status_connect_separator set to yes

With @catppuccin_status_connect_separator set to no

BEGIN_COMMIT_OVERRIDE
feat: replace _ctp_status_bg with unified _ctp_module_bg_color in module configuration
END_COMMIT_OVERRIDE

utils/status_module.conf

… the seperator to be visible

bvelastegui

Nice work!

kjnsn · 2024-10-19T01:34:23Z

Thanks so much! This will be released in v2

…ion to properly show module separation (catppuccin#380) * fix: module bg color blending in with status line bg color preventing the seperator to be visible --------- Co-authored-by: Kaley Main <kaleymain@google.com>

xafarr mentioned this pull request Oct 17, 2024

@catppuccin_status_connect_separator have no effect because of module background color blending with status-style background #381

Closed

1 task

kjnsn reviewed Oct 17, 2024

View reviewed changes

utils/status_module.conf Outdated Show resolved Hide resolved

fix: module bg color blending in with status line bg color preventing…

ef0665a

… the seperator to be visible

xafarr force-pushed the main branch from a53139e to ef0665a Compare October 17, 2024 09:57

bvelastegui reviewed Oct 18, 2024

View reviewed changes

fix: update tests for catppuccin#380

f7b114e

kjnsn merged commit a4d4ad0 into catppuccin:main Oct 19, 2024
1 of 2 checks passed

github-actions bot mentioned this pull request Oct 19, 2024

chore(main): release 2.0.0 #387

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: replace @_ctp_status_bg with @thm_surface_0 in module configurtion to properly show module separation #380

fix: replace @_ctp_status_bg with @thm_surface_0 in module configurtion to properly show module separation #380

xafarr commented Oct 17, 2024 •

edited by sgoudham

Loading

bvelastegui left a comment

kjnsn commented Oct 19, 2024

fix: replace @_ctp_status_bg with @thm_surface_0 in module configurtion to properly show module separation #380

fix: replace @_ctp_status_bg with @thm_surface_0 in module configurtion to properly show module separation #380

Conversation

xafarr commented Oct 17, 2024 • edited by sgoudham Loading

bvelastegui left a comment

Choose a reason for hiding this comment

kjnsn commented Oct 19, 2024

xafarr commented Oct 17, 2024 •

edited by sgoudham

Loading