Skip to content

AppManager/App improvements#80

Merged
alder merged 3 commits intomainfrom
frostmorn/refactor/app_appmanager
Feb 24, 2026
Merged

AppManager/App improvements#80
alder merged 3 commits intomainfrom
frostmorn/refactor/app_appmanager

Conversation

@frostmorn
Copy link
Collaborator

@frostmorn frostmorn commented Feb 24, 2026 •
edited
Loading

Feature list

  1. Control all App states from single synchronizing thread [appManager->loop] (Launch/Suspend/Deletion/Resume)
  2. Avoid User input passing to apps on App switching
  3. Optimizations of canvas recreation on app->flags changes
  4. Allow normal math without casts in app->setFlags()
  5. Ensure anyApp enters suspend mode on appManager->runApp
  6. Simplify App constructors, from now control canvas dimensions of a new app only via AppFlags

@frostmorn frostmorn added bug Something isn't working enhancement New feature or request labels Feb 24, 2026
black-ghost-off
black-ghost-off approved these changes Feb 24, 2026
View reviewed changes
Copy link
Member

@black-ghost-off black-ghost-off left a comment

Choose a reason for hiding this comment

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

LGTM

@alder alder merged commit a21df31 into main Feb 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alder alder alder approved these changes

@black-ghost-off black-ghost-off black-ghost-off approved these changes

Assignees

No one assigned

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@frostmorn @alder @black-ghost-off