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

Visible separator #233

Merged
merged 12 commits into from
Apr 21, 2024
Merged

Visible separator #233

merged 12 commits into from
Apr 21, 2024

Conversation

trigg
Copy link
Contributor

@trigg trigg commented Apr 16, 2024

In answer to #211

Allow for rendering a single thin line using primary color. Should be configurable from CSS and by default matchs system theme.

trigg added 3 commits April 16, 2024 01:50
@ammen99
Copy link
Member

ammen99 commented Apr 16, 2024

Wouldn't it be better to make it so that we support both spacingN and separatorN as widgets, so that the user can select which one to use where?

trigg added 3 commits April 16, 2024 10:57
…r for invisible spacing
Copy link
Member

@ammen99 ammen99 left a comment

Choose a reason for hiding this comment

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

LGTM, one small nit and we can merge.

trigg added 6 commits April 16, 2024 12:53
@mrsteve0924
Copy link

nice! cant wait to check it out.

Copy link
Member

@ammen99 ammen99 left a comment

Choose a reason for hiding this comment

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

Thanks!

@ammen99 ammen99 merged commit f857efb into WayfireWM:master Apr 21, 2024
2 checks passed
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.

None yet

3 participants