Skip to content

chore(ui): dedupe 10 double-exported assets and drop the dead system qrc - #319

Open
gmmcosta15 wants to merge 1 commit into
fix/resource-broken-refsfrom
chore/resource-dedupe
Open

chore(ui): dedupe 10 double-exported assets and drop the dead system qrc#319
gmmcosta15 wants to merge 1 commit into
fix/resource-broken-refsfrom
chore/resource-dedupe

Conversation

@gmmcosta15

Copy link
Copy Markdown
Collaborator

Description

  • Code refactor

  • Remove 10 assets exported by two .qrc files under two keys.

  • Delete system_resources.qrc and its blob, now fully redundant.

  • Repoint mainWindow, basicFilamentPanel, controlStackedWidget at the surviving keys.

  • Net 2156 lines removed.

Motivation

Two keys for identical bytes means the asset ships twice and caches twice at runtime.

Tests

make check green.

Screenshots

Pending on-device check: filament PLA/PETG/ABS buttons and the control-page fans button changed key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Refactor Enhancing code's readability, maintainability, and extensibility while addressing technical debt. UI/UX User Interface what changed with this code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant