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

fix: notify when clipboard utility is missing #2

Merged
rootiest merged 1 commits from fix/copy-fail into main 2026-04-29 02:18:27 +00:00
Conversation 0 Commits 1 Files Changed 1
+23 -3
rootiest commented 2026-04-29 02:17:51 +00:00
Owner
Copy Link
Copy Source

This PR adds system notifications and improved CLI feedback when the user attempts to copy a color but no clipboard utility (like wl-copy or xclip) is installed.

Changes

  • Added notify_error function for high-visibility warning notifications.
  • Updated process_color to capture clipboard failures and trigger notifications when --notify is active.
  • Refined the warning message for missing utilities to be more informative.

Verification

Verified using a restricted PATH environment where wl-copy and xclip were removed, confirming that notify-send is correctly invoked with a warning payload.

This PR adds system notifications and improved CLI feedback when the user attempts to copy a color but no clipboard utility (like wl-copy or xclip) is installed. ### Changes - Added `notify_error` function for high-visibility warning notifications. - Updated `process_color` to capture clipboard failures and trigger notifications when `--notify` is active. - Refined the warning message for missing utilities to be more informative. ### Verification Verified using a restricted `PATH` environment where `wl-copy` and `xclip` were removed, confirming that `notify-send` is correctly invoked with a warning payload.
rootiest added 1 commit 2026-04-29 02:17:51 +00:00
fix: notify when clipboard utility is missing
Release on Merge / release (pull_request) Successful in 6s
Details
5f659b90e3 
- Add notify_error function to send normal urgency warning notifications
- Update process_color to track clipboard failure and trigger error notification
- Improve warning message when clipboard utilities (wl-copy/xclip) are absent
rootiest merged commit 344fc140dc into main 2026-04-29 02:18:27 +00:00
rootiest deleted branch fix/copy-fail 2026-04-29 02:18:27 +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 Label
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#2
Delete Branch "fix/copy-fail"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?