fix(imagebutton): tiling regression

[liamHowatt]

Description of the feature or fix

A regression was introduced by #6029. The lv_imagebutton does not set the image_area field of the lv_draw_image_dsc_t so it still holds the default value set by lv_draw_image_dsc_init of {0, 0, LV_COORD_MIN, 0} when it reaches _lv_draw_image_tiled_helper.

If draw_dsc->image_area is the invalid, fallback on coords which is what it used to always use before #6029. What I do not know is whether this fix will undo the fix done by #6029. @kisvegabor are you able to determine this? Thanks.

Added a test so imagebutton does not break again.

diff --git a/src/draw/lv_draw_image.c b/src/draw/lv_draw_image.c
index 263260613..b4ffb9ad5 100644
--- a/src/draw/lv_draw_image.c
+++ b/src/draw/lv_draw_image.c
@@ -185,7 +185,13 @@ void _lv_draw_image_tiled_helper(lv_draw_unit_t * draw_unit, const lv_draw_image
     int32_t img_w = draw_dsc->header.w;
     int32_t img_h = draw_dsc->header.h;
 
-    lv_area_t tile_area = draw_dsc->image_area;
+    lv_area_t tile_area;
+    if(lv_area_get_width(&draw_dsc->image_area) >= 0) {
+        tile_area = draw_dsc->image_area;
+    }
+    else {
+        tile_area = *coords;
+    }
     lv_area_set_width(&tile_area, img_w);
     lv_area_set_height(&tile_area, img_h);

At the time of creating this PR:

Notes

• Update the Documentation if needed.
• Add Examples if relevant.
• Add Tests if applicable.
• If you added new options to lv_conf_template.h run lv_conf_internal_gen.py and update Kconfig.
• Run scripts/code-format.py (astyle version v3.4.12 needs to be installed) and follow the Code Conventions.
• Mark the Pull request as Draft while you are working on the first version, and mark is as Ready when it's ready for review.
• When changes were requested, re-request review to notify the maintainers.
• Help us to review this Pull Request! Anyone can approve or request changes.

[kisvegabor]

Awesome! Special thanks for the test! 🙂