NotificationStack: Check if notification was already shown

[lenemter]

After a reboot i cannot reproduce #1783 anymore, so this is my first wild guess on what might be going on.

[lenemter]

@TomiOhl Can you check if it makes any difference on your machine please? I can give instructions if you need them.

[TomiOhl]

@TomiOhl Can you check if it makes any difference on your machine please? I can give instructions if you need them.

I've also rebooted and currently I cannot reproduce it either 😅 Not sure what triggers the behavior. Would be useful to know in order to test properly.
Instructions: the ones in the readme are up-to-date?

[lenemter]

Instructions: the ones in the readme are up-to-date?

Yes, they are. You can quickly install build dependencies using sudo apt build-dep gala

[TomiOhl]

Cool, thanks for the tip. I can try it on my other laptop in the evening (approx. 6h from now).

[TomiOhl]

So I've tried to reproduce the issue on my other laptop, but I couldn't do so neither on this branch nor on master. Would be useful to know what causes it really.
Things that I haven't tried here but do cause other issues in gala:

• multiple displays
• different aspect ratios

I did get the issue again on my work laptop today though. I don't really want to use it for testing but right now I feel like that would be the easiest thing that I could do on my end

[lenemter]

I think we should merge this branch, since the change is very small and won't introduce any new issues, but might fix the notifications issue.