Skip to content

Comments

Add output and labels to quickcmd#1534

Open
pajawojciech wants to merge 10 commits intoDFHack:masterfrom
pajawojciech:quickcmd-output-and-labels
Open

Add output and labels to quickcmd#1534
pajawojciech wants to merge 10 commits intoDFHack:masterfrom
pajawojciech:quickcmd-output-and-labels

Conversation

@pajawojciech
Copy link
Contributor

@pajawojciech pajawojciech commented Jan 8, 2026

Changes

  • Custom names for commands, editable via Shift+N
  • When name is not empty then it shows instead of command
  • Checkbox to display command output in a popup window (Shift+O)
  • Config data is stored in separate config file because data structure changed from string array to objects array (in quickcmd-v2.json instead of quickcmd.json)
  • If there is no data in new config file then auto migrate from old config

quickcmd window:

obraz

Output window:

obraz

Old and new settings file:

obraz

Issues

@ab9rf ab9rf added this to 53.09-r2 Jan 8, 2026
@github-project-automation github-project-automation bot moved this to Todo in 53.09-r2 Jan 8, 2026
@ab9rf ab9rf moved this from Todo to Being worked on in 53.09-r2 Jan 8, 2026
@ab9rf ab9rf removed this from 53.09-r2 Jan 12, 2026
@ab9rf ab9rf added this to 53.10-r2 Jan 12, 2026
@github-project-automation github-project-automation bot moved this to Todo in 53.10-r2 Jan 12, 2026
@ab9rf ab9rf moved this from Todo to Being worked on in 53.10-r2 Jan 12, 2026
@ab9rf ab9rf requested a review from chdoc February 9, 2026 15:11
@ab9rf ab9rf moved this from Being worked on to Review In Progress in 53.10-r2 Feb 9, 2026
@chdoc chdoc self-requested a review February 10, 2026 18:26
Copy link
Member

@chdoc chdoc left a comment

Choose a reason for hiding this comment

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

Overall, I think this is a pretty nice change and the code looks pretty clean. Please update the configuration filed behavior as discussed in my previous comment and use a canonical value for commands that do not have assigned names.

@ab9rf ab9rf moved this from Review In Progress to Needs revision in 53.10-r2 Feb 16, 2026
@pajawojciech pajawojciech requested a review from chdoc February 21, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs revision

Development

Successfully merging this pull request may close these issues.

3 participants