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

fix(nema): Broken build after draw unit refactoring, draw_label changes #7759

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

Conversation

MarkIoanAmat
Copy link
Contributor

Fixes #7691

Notes

uLipe
uLipe previously approved these changes Feb 12, 2025
@MarkIoanAmat
Copy link
Contributor Author

@liamHowatt @uLipe @kisvegabor
I also added some new color formats and the fix for a glitching bug found in e-bike (cant find the ticket though) I think it was initially reported by @tdjastrzebski

Also fixes #7723

Copy link
Member

@uLipe uLipe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work I have just two small nits :)

src/draw/nema_gfx/lv_draw_nema_gfx_label.c Outdated Show resolved Hide resolved
src/draw/nema_gfx/lv_draw_nema_gfx_label.c Outdated Show resolved Hide resolved
@MarkIoanAmat
Copy link
Contributor Author

Also found ticket for e-bike glitch, was #7439

@kisvegabor
Copy link
Member

Thank you! @uLipe when you will have some spare time, please check Dave2D and update it if needed.

@kisvegabor
Copy link
Member

Also found ticket for e-bike glitch, was #7439

Will you send a PR for this?

@MarkIoanAmat
Copy link
Contributor Author

@kisvegabor It is fixed in this PR

@MarkIoanAmat MarkIoanAmat requested a review from uLipe February 14, 2025 10:10
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.

Broken build on lv_draw_nema_gfx.c after 'refactor(draw): use task instead of draw_unit'
3 participants