Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pressing a mouse button which isn't one of configured ones results in a crash #101

Closed
balsoft opened this issue Oct 31, 2020 · 0 comments · Fixed by #102
Closed

Pressing a mouse button which isn't one of configured ones results in a crash #101

balsoft opened this issue Oct 31, 2020 · 0 comments · Fixed by #102

Comments

@balsoft
Copy link
Contributor

balsoft commented Oct 31, 2020

With a default configuration, if I press a middle mouse button on a notification, it results in deadd-notification-center crashing with the following message:

deadd-notification-center: src/NotificationCenter/Notifications/NotificationPopup.hs:(138,5)-(156,18): Non-exhaustive guards in multi-way if

Related: #99 (comment)

balsoft added a commit to balsoft/linux_notification_center that referenced this issue Oct 31, 2020
Problem: button clicks which aren't bound in a configuration file to either
dismiss or perform an action on a notification crash the notification center

Solution: add a catch-all clause to mouse button handling routine, closing
the notification if an unknown button press is received.

Closes phuhl#101
@phuhl phuhl closed this as completed in #102 Nov 1, 2020
ahmubashshir pushed a commit to ahm-forks/linux_notification_center that referenced this issue Nov 7, 2020
Problem: button clicks which aren't bound in a configuration file to either
dismiss or perform an action on a notification crash the notification center

Solution: add a catch-all clause to mouse button handling routine, closing
the notification if an unknown button press is received.

Closes phuhl#101
ahmubashshir pushed a commit to ahm-forks/linux_notification_center that referenced this issue Nov 7, 2020
Problem: button clicks which aren't bound in a configuration file to either
dismiss or perform an action on a notification crash the notification center

Solution: add a catch-all clause to mouse button handling routine, closing
the notification if an unknown button press is received.

Closes phuhl#101
avdv pushed a commit to avdv/linux_notification_center that referenced this issue Dec 14, 2020
Problem: button clicks which aren't bound in a configuration file to either
dismiss or perform an action on a notification crash the notification center

Solution: add a catch-all clause to mouse button handling routine, closing
the notification if an unknown button press is received.

Closes phuhl#101
ahmubashshir pushed a commit to ahm-forks/linux_notification_center that referenced this issue Jan 22, 2021
Problem: button clicks which aren't bound in a configuration file to either
dismiss or perform an action on a notification crash the notification center

Solution: add a catch-all clause to mouse button handling routine, closing
the notification if an unknown button press is received.

Closes phuhl#101


Former-commit-id: 795d66c
ahmubashshir pushed a commit to ahm-forks/linux_notification_center that referenced this issue May 5, 2021
Problem: button clicks which aren't bound in a configuration file to either
dismiss or perform an action on a notification crash the notification center

Solution: add a catch-all clause to mouse button handling routine, closing
the notification if an unknown button press is received.

Closes phuhl#101


Former-commit-id: 1158c3bacdde935230702313b44f862a1f08bb38 [formerly 95b69bd] [formerly 95b69bd [formerly 95b69bd [formerly 795d66c]]]
Former-commit-id: a6af0caabc4d49f83be5bff10b1bc574ee98a751
Former-commit-id: ffe554430b05e1d6fdfa984b9a801acfdf13c56d
Former-commit-id: 5435759846f9d103222958d27ee0e275240743cc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant