wired-notify icon indicating copy to clipboard operation
wired-notify copied to clipboard

[BUG] Trailing white space in the notification body

Open faynopi opened this issue 3 years ago • 1 comments

Version: 0.10.2 - wired-git from aur OS: Arch Linux + dwm

  • [x] I am running the latest version
  • [x] I checked the documentation and found no answer
  • [x] I checked to make sure that this issue has not already been filed

Expected Behavior

I think the trailing spaces should be removed before sending the notification.

Current Behavior

image This is a Brave browser advertising with just an empty space in the body.

Steps to Reproduce

image

notify-send "test" "   "
notify-send "test" " "

faynopi avatar Aug 26 '22 17:08 faynopi

I think I considered this at some point and decided it's a bug with the notification sender, but I'm happy to change my mind.

An alternative is changing the Body RenderCriteria to ignore whitespace in the criteria, but I don't see this being particularly useful.

I think just trimming notification text is the right idea.

Toqozz avatar Aug 28 '22 09:08 Toqozz

Fixed in 0.10.3.

Toqozz avatar Jan 13 '24 04:01 Toqozz