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

chwb3 #40

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

chwb3 #40

wants to merge 4 commits into from

Conversation

Sweets
Copy link

@Sweets Sweets commented Aug 23, 2018

Adds chwb3, my fork of chwb2 (that was almost 3.5 years past due) for making three window borders (really still two, the outer borders are the same color and size as each other).

@Sweets Sweets changed the title chwb3 - Triple borders chwb3 Aug 23, 2018
@neeasade
Copy link

suggestion: maybe at this point make it chwbn, with an arg to set the border colors/widths? .

something like chwbn -n 3 -c <color> -c <color> -c <color> -w 2 -w 4 -w 2 <wids>, where each -c and -w sets the next border from inside out or vice versa.

@Sweets
Copy link
Author

Sweets commented Aug 23, 2018

@neeasade I'd say that suggestion may work for borders >= 4, but for just chwb3, chwb2, and chwb, I'd say it wouldn't mesh well.

Especially considering that chwb and chwb2 are in two completely different repositories, users could potentially have chwb, but not chwb2, and having chwbn would be completely useless to that demographic of users (and would take up space on their drive, moreso than is necessary).

More than 3 borders seems unnecessary, as well. Good suggestion, though, I just don't think it would apply very well in this scenario.

neeasade referenced this pull request in neeasade/opt Aug 28, 2018
@z3bra
Copy link
Contributor

z3bra commented Nov 16, 2018

I would be pretty redundant to have 2 tools aside that do the exact same thing (draw a pixmap on thw borders).
As neasade said, it would be best to turn chwb2 into a more generic purpose tool, eg. dwpb for "draw pixmap on window borders" or something along those lines.

because what a user see as "unnecessary number of borders" is arbitrary (and to me, more than 1 border seem unnecessary) ;)

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.

3 participants