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

Refactoring structure of the launcher app menu #158

Merged
merged 4 commits into from
Dec 27, 2024

Conversation

drronnie85
Copy link
Collaborator

#155

Ability use B as back button in the menu.
Every menu item has a submenu or callback func.
Also added update method to the menu to update the menu items (postfix in WiFi item: ON/OFF).

Ability use B as back button in the menu.
Every menu item has a submenu or callback func.
Also added update method to the menu to update the
menu items (postfix in WiFi item: ON/OFF).
@drronnie85 drronnie85 changed the title Refactoring structure of the launcher app menu #Draft Refactoring structure of the launcher app menu Dec 26, 2024
@drronnie85 drronnie85 changed the title #Draft Refactoring structure of the launcher app menu Refactoring structure of the launcher app menu Dec 26, 2024
This reverts commit 7c0800a.
@drronnie85
Copy link
Collaborator Author

Упс. Випадково залив кривий statusbar.cpp.
Ревертнув

AppManager::getInstance()->runApp(list[index].construct());
menu_icon_t* icon = list[i].icon;
if (icon == NULL) {
icon = list[i].submenu.empty() ? &app_img : &app_group_img;
Copy link
Collaborator

Choose a reason for hiding this comment

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

вот, через це в налаштуваннях і таблиці розділів тепер є іконки яких не було :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Лол) Було відчуття що щось не так) Але не зрозумів, що саме.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Виправлено

@frostmorn
Copy link
Collaborator

а загалом норм

Copy link
Collaborator

@frostmorn frostmorn left a comment

Choose a reason for hiding this comment

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

LGTM. Tested on hardware

@frostmorn frostmorn merged commit 5a6d90a into and3rson:main Dec 27, 2024
3 checks passed
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.

2 participants