Gustash
On branch notify-icon-fix
Changes to be committed: modified: hyprshot
Addressed the following issues:
-
Allow the user to leverage their system settings for the timeout in
notify-send. Previously, the timeout was defaulting to 5000 ms, which is quite long for an information message. While it could be changed, it was not responsive to or respecting system settings. -
Fix the icon for the notifications. The information icon was not displayed when a screenshot was taken since the priority of the message was not given as an argument to
notify-send. By adding the 'dialog-information' priority, the icon is displayed conditionally when the screenshot is taken and not saved to a file. When saved to a file, the screenshot is used as the icon. -
Improved readability of the notifications title and body text. The name of the tool, Hyprshot, is now displayed in the title, and the body includes clarifying text indicating the screenshot was taken rather than the more generic verbiage about an image being saved.
After: Priority associated icon is present, and it's properly registering as an informational message.
Before: Priority inappropriately registering as normal rather that informational, and lacking the priority associated icon in absence of the image being saved to disk.
