-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Modify the appearance for option and checkbox! #653
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: zhouxs1023 <zhouxs1023@163.com>
@@ -15838,13 +15838,12 @@ nk_draw_checkbox(struct nk_command_buffer *out, | |||
|
|||
/* draw background and cursor */ | |||
if (background->type == NK_STYLE_ITEM_COLOR) { | |||
nk_fill_rect(out, *selector, 0, style->border_color); | |||
nk_fill_rect(out, nk_shrink_rect(*selector, style->border), 0, background->data.color); | |||
nk_stroke_rect(out, *selector, 2, 2, background->data.color); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a total nitpicking, but this indentation is not correct 😉.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have looked at the codes ,It are correct! I had updated pics!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The picture is totally fine. It's just, that this line is indented with tabs, but shall be indented with spaces. Otherwise I think this commit can get merged.
@@ -15838,13 +15838,12 @@ nk_draw_checkbox(struct nk_command_buffer *out, | |||
|
|||
/* draw background and cursor */ | |||
if (background->type == NK_STYLE_ITEM_COLOR) { | |||
nk_fill_rect(out, *selector, 0, style->border_color); | |||
nk_fill_rect(out, nk_shrink_rect(*selector, style->border), 0, background->data.color); | |||
nk_stroke_rect(out, *selector, 2, 2, background->data.color); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The picture is totally fine. It's just, that this line is indented with tabs, but shall be indented with spaces. Otherwise I think this commit can get merged.
@zhouxs1023 could you change all the pictures in the main README.md file which show the radio buttons and are not using bitmap styling? |
Signed-off-by: zhouxs1023 zhouxs1023@163.com
Modify the appearance for option and checkbox!