added safety guard to not open image browser until app is not ready
This commit is contained in:
@@ -10,4 +10,5 @@ typedef enum {
|
|||||||
void create_log_ui(void);
|
void create_log_ui(void);
|
||||||
void create_image_browser_ui(void);
|
void create_image_browser_ui(void);
|
||||||
|
|
||||||
|
void ui_app_ready_for_image_browser(void);
|
||||||
void ui_switch_view_to(ui_view_t view);
|
void ui_switch_view_to(ui_view_t view);
|
||||||
@@ -78,6 +78,12 @@ void create_image_browser_ui(void){
|
|||||||
ui_log_appendf("Image browser UI created with %u images", image_count);
|
ui_log_appendf("Image browser UI created with %u images", image_count);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static bool app_ready_for_image_browser = false;
|
||||||
|
|
||||||
|
void ui_app_ready_for_image_browser(void){
|
||||||
|
app_ready_for_image_browser = true;
|
||||||
|
}
|
||||||
|
|
||||||
void ui_switch_view_to(ui_view_t view){
|
void ui_switch_view_to(ui_view_t view){
|
||||||
if(view == current_view) return;
|
if(view == current_view) return;
|
||||||
|
|
||||||
@@ -88,6 +94,11 @@ void ui_switch_view_to(ui_view_t view){
|
|||||||
lvgl_port_unlock();
|
lvgl_port_unlock();
|
||||||
break;
|
break;
|
||||||
case UI_VIEW_IMAGE_BROWSER:
|
case UI_VIEW_IMAGE_BROWSER:
|
||||||
|
if(!app_ready_for_image_browser){
|
||||||
|
ESP_LOGW(TAG, "Cannot switch to image browser view, app not ready");
|
||||||
|
ui_log_append("Cannot switch to image browser view, app not ready");
|
||||||
|
return;
|
||||||
|
}
|
||||||
lvgl_port_lock(0);
|
lvgl_port_lock(0);
|
||||||
ui_build_image_browser_view_locked();
|
ui_build_image_browser_view_locked();
|
||||||
lvgl_port_unlock();
|
lvgl_port_unlock();
|
||||||
|
|||||||
@@ -37,6 +37,7 @@ void app_main(void){
|
|||||||
}
|
}
|
||||||
|
|
||||||
ui_log_append("Done listing files in root directory");
|
ui_log_append("Done listing files in root directory");
|
||||||
|
ui_app_ready_for_image_browser();
|
||||||
|
|
||||||
ui_log_append("\nSwitching to image browser UI");
|
ui_log_append("\nSwitching to image browser UI");
|
||||||
ESP_LOGI(TAG, "Switching to image browser UI");
|
ESP_LOGI(TAG, "Switching to image browser UI");
|
||||||
|
|||||||
Reference in New Issue
Block a user