rootiest/color-tool
1
1
Fork 0
Code Issues Pull Requests Actions Releases 6 Activity

feat(config): enable desktop swatch and improve comment alignment #11

Merged
rootiest merged 2 commits from feat/desktop-swatch-and-config-alignment into main 2026-05-03 18:58:03 +00:00
Conversation 0 Commits 2 Files Changed 2
+30 -3
rootiest commented 2026-05-03 18:57:16 +00:00
Owner
Copy Link
Copy Source

This PR introduces two minor improvements to the configuration management:

  1. Default Swatch for Desktop Mode: Sets swatch = true as the default in the stock [desktop] configuration and internal defaults. This ensures that desktop notifications include a visual color swatch by default, improving the user experience for the app-menu launcher.
  2. Comment Alignment in set_config: Implements a robust whitespace adjustment logic in the configuration updater. When toggling between values of different lengths (like true and false), the tool now dynamically adds or removes spaces to keep trailing comments vertically aligned in config.toml.

Both changes have been verified with manual reproduction scripts and the existing test suite.

This PR introduces two minor improvements to the configuration management: 1. **Default Swatch for Desktop Mode**: Sets `swatch = true` as the default in the stock `[desktop]` configuration and internal defaults. This ensures that desktop notifications include a visual color swatch by default, improving the user experience for the app-menu launcher. 2. **Comment Alignment in `set_config`**: Implements a robust whitespace adjustment logic in the configuration updater. When toggling between values of different lengths (like `true` and `false`), the tool now dynamically adds or removes spaces to keep trailing comments vertically aligned in `config.toml`. Both changes have been verified with manual reproduction scripts and the existing test suite.
rootiest added 2 commits 2026-05-03 18:57:16 +00:00
feat(config): enable swatch by default for desktop mode 8d94bc4006 
Set swatch = true as the default in the stock [desktop] configuration
and internal defaults. This ensures that desktop notifications include
a visual color swatch by default. Updated README and tests accordingly.
feat(config): maintain comment alignment in set_config
Auto Label PRs / label (pull_request) Successful in 3s
Details
Test PR / test (pull_request) Successful in 6s
Details
Release on Merge / release (pull_request) Successful in 6s
Details
e0967a8388 
Updated set_config to dynamically adjust leading whitespace of trailing
comments when toggling between true/false (or any values of different
lengths). This ensures that inline comments in config.toml remain
vertically aligned after programmatic updates.
rootiest merged commit 2b58cc8a13 into main 2026-05-03 18:58:03 +00:00
rootiest deleted branch feat/desktop-swatch-and-config-alignment 2026-05-03 18:58:03 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
rootiest (Rootiest)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: rootiest/color-tool#11