Skip to content

Conversation

@pajawojciech
Copy link

@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
Comment on lines +37 to +38
local CONFIG_FILE_OLD = 'dfhack-config/quickcmd.json'
local CONFIG_FILE = 'dfhack-config/quickcmd-v2.json'
Copy link
Member

Choose a reason for hiding this comment

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

@ab9rf Is there a policy for updating the configuration format for tools that are not world or site specific? I would tend to keep the configuration filename aligned with the name of the tool without adding any version suffixes, putting any versioning information about the configuration file format into the configuration file itself. That is, back up the old configuration file under a new name and then convert the configuration in place.

Copy link
Member

Choose a reason for hiding this comment

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

I'm inclined to agree with you here: keep the file name the same and store the version information into the file itself.

@chdoc chdoc self-requested a review February 10, 2026 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Review In Progress

Development

Successfully merging this pull request may close these issues.

3 participants