continuation of #468 : checkbox radio button appearance update #681
Conversation
Verified that toggle boxes and buttons look good for all SDL, X11, and GLFW demos. Also respects different styles, which is the biggest change with the recent merge with master.
|
Thanks for revamping this. I like the direction, and it looks great on the OpenGL demos, but on some of the other renderers, it looks a bit off, and I'm not entirely sure why it's happening... X11Before
After
RawFB SDLBefore
After
|
|
i took a look into the X11 example and I cant find anything in specific that would be causing this. It could be something particular with how the X11 example is handling the draw call. However, your SDL example is interesting as both seem to have very similar problems. I will take another look to see if i can't find anything. However, i must ask. is this a show stopper? would the slight variation in the render out weight the benefit of the overall change? My personal take is, no and that clarity of the selection is more important. |
The new checkbox toggle look was off by a couple of pixels. This commit fixes that by removing the "line thickness" operation performed on its width before drawing.
The new checkbox toggle look was off by a couple of pixels. This commit fixes that by removing the "line thickness" operation performed on its width before drawing.
I figured out the issue with the X11 demos and have provided a fix in this PR. However, There seems to be some different kind of issue with RAW_FB stuff and that I haven't been able to solve yet. This doesn't seem to be a problem with the toggle code, more of a draw issue specific to RAW_FB. BEFORE
After
|
|
Great find. Thanks for the updates. Do you think this is ready enough to merge in? |
|
Definitely good to go |
|
Hmm, still looks off a bit on some of the other renderers. Perhaps we keep the radios as circles, and the checkboxes as non-rounded boxes? 🤔 I did see that some of the parent data is being set in |
My understanding is that the checkbox is drawing a rect_stroke, then the do_toggle is doing a rect_fill of a smaller size to overlap the inside of the checkbox. But for it still looking "off" on some. The basic XDraw commands on rounded edges are producing the artifacts regardless of the new toggle behaviour. I think removing the rounded corners for checkboxes would definitely help. |
the rounded corners on the checkboxes looked terrible with the X11 and rawfb demos. This was also in-congruent with the previous look. Reverting to sharp corners keeps the previous look while also removing the issue of the X11 looking terrible. This commit also shrinks the select toggle box inside the checkbox by a couple pixels to better match the radio button's look and feel
|
The corner issues have been fixed. X11 and others now look great. I believe this PR is ready to go.
|
All credit goes to those referenced in other PRs (#468) . I have updated the original branch to remove conflicts with master. I have also verified the checkbox appearance with all SDL, GLFW, X11, and some GDI. All of the different styles seem to handle the update nicely.
Note: this might not be the best way to update things; but i couldn't push to the original and I figured that a 2 year old branch might not get much attention from the original author even if I did ask for a PR into that branch.
Please let me know if there is anything that I can do, as I really want to see this specific update get integrated