bluerobotics / bluerobotics/cockpit

We should decouple the logging system from the widget management system

Open
#939 1 comment 0 reactions 0 assignees View on GitHub
enhancement refactor
Dominant language
TypeScript
Stars
198
Forks
63
Avg merge
2d 7h
Merged PRs (30d)
57

Description

On PR #913 we added a coupling between the logging system and the widget management system.
We did that as it was a quicker solution, so we don't hurt the stable roadmap, as there are other PRs waiting for this one.

We should try to decouple them again.

One of the ideas to solve this is to allow register of add/remove callbacks on the widget manager, so those interested can react to changes there. This can also be accomplished through the reactivity system (proxys).

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing PR #913 and tracing how the logging system currently depends on widget management. Compare the proposed widget-manager callbacks with the reactivity-system option; done means the two systems no longer depend directly on each other while widget add/remove changes still reach interested logging code.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.