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

Rounded corner cleanup #480

Closed
wants to merge 3 commits into from
Closed

Rounded corner cleanup #480

wants to merge 3 commits into from

Conversation

yshui
Copy link
Owner

@yshui yshui commented Aug 31, 2020

No description provided.

@yshui
Copy link
Owner Author

yshui commented Sep 1, 2020

ping #361 and #229

@yshui yshui force-pushed the round branch 6 times, most recently from 0210855 to ed2b16f Compare September 5, 2020 15:59
evanjs added a commit to evanjs/nixos_cfg that referenced this pull request Oct 14, 2020
Use yshui/picom#361 until
yshui/picom#480 is merged

kitty: use spacecamp-kitty theme
xmonad: enable gaps
@jslick
Copy link

jslick commented Oct 24, 2020

Hi, is this branch a WIP? I'm having trouble building it.

[16/30] Compiling C object src/picom.p/backend_backend.c.o
FAILED: src/picom.p/backend_backend.c.o
cc -Isrc/picom.p -Isrc -I../src -Isubprojects/test.h -I../subprojects/test.h -I/usr/include/pixman-1 -I/usr/include/dbus-1.0 -I/usr/lib/dbus-1.0/include -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -std=c11 -O3 '-DCOMPTON_VERSION="vgit-2c696"' -DNDEBUG -D_GNU_SOURCE -DHAS_STDC_PREDEF_H -Wall -Wcast-function-type -Wignored-qualifiers -Wmissing-parameter-type -Wnonnull -Wshadow -Wno-type-limits -Wold-style-declaration -Woverride-init -Wsign-compare -Wtype-limits -Wuninitialized -Wshift-negative-value -Wunused-but-set-parameter -Wunused-parameter -Wimplicit-fallthrough -Wno-missing-braces -Wconversion -Wempty-body -march=x86-64 -mtune=generic -O2 -fno-plt -D_FORTIFY_SOURCE=2 -DCONFIG_LIBCONFIG -DCONFIG_REGEX_PCRE -DCONFIG_REGEX_PCRE_JIT -DCONFIG_OPENGL -DGL_GLEXT_PROTOTYPES -DCONFIG_DBUS -DHAS_INOTIFY -MD -MQ src/picom.p/backend_backend.c.o -MF src/picom.p/backend_backend.c.o.d -o src/picom.p/backend_backend.c.o -c ../src/backend/backend.c
../src/backend/backend.c: In function ‘paint_all_new’:
../src/backend/backend.c:152:57: warning: passing argument 3 of ‘ps->backend_data->ops->compose’ makes integer from pointer without a cast [-Wint-conversion]
  152 |   ps->backend_data->ops->compose(ps->backend_data, t, ps->root_image, 0, 0,
      |                                                       ~~^~~~~~~~~~~~
      |                                                         |
      |                                                         void *
../src/backend/backend.c:152:57: note: expected ‘int’ but argument is of type ‘void *’
../src/backend/backend.c:152:3: error: too many arguments to function ‘ps->backend_data->ops->compose’
  152 |   ps->backend_data->ops->compose(ps->backend_data, t, ps->root_image, 0, 0,
      |   ^~

@tryone144
Copy link
Collaborator

tryone144 commented Oct 24, 2020

Hi, is this branch a WIP? I'm having trouble building it.

Yup, see #493 (comment):

Thanks for your contribution. The round branch is still WIP so it's not expected to compile.

@subnut
Copy link

subnut commented Oct 28, 2020

@yshui Any ETA to get it done?
I mean, ibhagwan's fork is working fine, isn't it? So why not here?

ibhagwan and others added 3 commits December 17, 2020 23:21
wip: removed old shader, rounded borders almost perfect

rounded border WIP

Fix a type error in the rounded corners shader

remove one of the shader?

x
@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #480 (585e406) into next (a335eed) will decrease coverage by 0.27%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             next     #480      +/-   ##
==========================================
- Coverage   37.51%   37.23%   -0.28%     
==========================================
  Files          46       46              
  Lines        9432     9456      +24     
==========================================
- Hits         3538     3521      -17     
- Misses       5894     5935      +41     
Impacted Files Coverage Δ
src/opengl.c 0.00% <0.00%> (ø)
src/opengl.h 60.46% <ø> (ø)
src/win.c 68.07% <0.00%> (-0.98%) ⬇️
src/win.h 78.12% <ø> (ø)
src/picom.c 67.54% <0.00%> (-0.64%) ⬇️
src/backend/dummy/dummy.c 80.32% <0.00%> (-0.32%) ⬇️

@RandomExplosion
Copy link

@yshui what's the current status on this PR?

@tryone144
Copy link
Collaborator

This PR has been closed in favour of #551, #558, and #614 for the legacy backends. Rounded corners should already work there.
The implementation of rounded corners for the experimental-backends is tracked in #658 (currently WIP).

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.

6 participants