lc-soft / LCUI

C library for building user interfaces
https://lcui-dev.github.io
MIT License
4.13k stars 355 forks source link

Improve highlighting of repainted areas #180

Closed lc-soft closed 4 years ago

lc-soft commented 4 years ago

Issuehunt badges

Is your feature request related to a problem? Please describe. https://github.com/lc-soft/LCUI/blob/4a9516e8eb3bc23b681575653ac31468093d8a2c/src/display.c#L173-L184

Describe the solution you'd like

Remove DrawBorder() function and use FlashPaintArea() instead, this function should do the following:

The final effect should be the same as the paint flashing of Google Chrome:

highlights-areas

Describe alternatives you've considered None.

Additional context None.


IssueHunt Summary #### [vbalyasnyy vbalyasnyy](https://issuehunt.io/u/vbalyasnyy) has been rewarded. ### Backers (Total: $10.00) - [lc-soft lc-soft](https://issuehunt.io/u/lc-soft) ($10.00) ### Submitted pull Requests - [#190 feat(gui): flashing rendered rects (#180)](https://issuehunt.io/r/lc-soft/LCUI/pull/190) --- ### Tips - Checkout the [Issuehunt explorer](https://issuehunt.io/r/lc-soft/LCUI/) to discover more funded issues. - Need some help from other developers? [Add your repositories](https://issuehunt.io/r/new) on IssueHunt to raise funds. --- IssueHunt has been backed by the following sponsors. [Become a sponsor](https://issuehunt.io/membership/members)
issuehunt-oss[bot] commented 4 years ago

@lc-soft has funded $5.00 to this issue.


anshulxyz commented 4 years ago

hi, @lc-soft can you describe further what you mean by "improve" the highlighting, can you show some examples?

issuehunt-oss[bot] commented 4 years ago

@lc-soft has funded $5.00 to this issue.


anshulxyz commented 4 years ago

@lc-soft do you have any design/style in mind?

lc-soft commented 4 years ago

@anshulxyz This is a low priority task and I will update the description as soon as possible.

anshulxyz commented 4 years ago

Alright :pray:

lc-soft commented 4 years ago

@anshulxyz The issue description has been updated. Maybe the implementation of this feature is a bit complicated.

issuehunt-oss[bot] commented 4 years ago

@lc-soft has rewarded $8.00 to @vbalyasnyy. See it on IssueHunt