ARTIST-Association / ARTIST-Association/CANVAS

Reduce duplication for the command prompt commands

Open
#60 0 comments 0 reactions 0 assignees View on GitHub
code quality
Dominant language
Python
Stars
6
Forks
2
PR merge metrics
No merged PRs in 30d

Description

The `promptCommands.mjs` often reimplement existing functionality to execute in the command. Consider reusing already implemented code

Examples:
- the `ToggleFullscreenPromptCommand` overlaps with the fullscreen button `setupFullscreen` in `Navbar`
- the `ThemePromptCommand` overlaps with `darkmode.js`. Are both necessary or can we reuse code?

Contributor guide

Open the contributing guide

Research direction

Start by reading promptCommands.mjs, then compare ToggleFullscreenPromptCommand with Navbar's setupFullscreen and ThemePromptCommand with darkmode.js. Identify which existing functionality can be shared without changing command behavior; done means the duplicated implementations are reduced and the relevant prompt commands still work.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.