Skip to content

Code Quality: Use RichCommand for widget items #14760

Open
@0x5bfa

Description

@0x5bfa

Description

Widget pages have a very complicated codebase such as hooking events in Home page triggered in each widgets. I think this is because of associatedInstance but we can simplify it. In my search, achieving this goal will reduce about 500 lines. To be simple, I splitted up the work.

Concerned code

  • Widget controls
  • Home page
  • Actions

Gains

  • Far better maintainability

Requirements

Stage 1 (Done) (approx. 1200 lines)

  • Introduce view models to widget controls

Stage 2 (Done) (approx. 150 lines)

  • Remove events in Home and let widget view models handle them

Stage 3 (In Progress) (approx. 500 lines)

  • Replace ICommands with RichCommands

Stage 4 (Ready) (approx. 100 lines)

  • Fix some bugs around refresh button and automatic refresh

Comments

No response

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

Status

🏗 In progress

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions