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

Monitor #20

Closed
plasmoduck opened this issue Nov 15, 2020 · 5 comments · May be fixed by #21
Closed

Monitor #20

plasmoduck opened this issue Nov 15, 2020 · 5 comments · May be fixed by #21

Comments

@plasmoduck
Copy link

How can I select what monitor to display notifcations? I have 2 monitors and I'd like it on monitor 1 but it shows on monitor 2.

@dudik
Copy link
Owner

dudik commented Nov 15, 2020

Right now there's no way, but it should be pretty straightforward to implement. Sadly, I'm still in the middle of a somewhat busy semester and I don't have much free time. I will gladly implement this, but not sooner than 2-3 months from now. I'm really sorry.

@TAAPArthur
Copy link

I can submit a patch. I would lean towards adding a patch that used the primary monitor's size instead of the default screen dimensions. For help setting the primary monitor see xrandr

Rationale:
As far as I'm aware, monitor ids/numbers (and possibly even name depending on your setup) aren't useful for static configurations as they can change on restart/re-plug. Also it seem to be common to in other notification systems.

Let me know if this doesn't work for your use case.

@MostHated
Copy link

Was this added in? I don't see the option for it in the config.h file. Does it need to be manually added? I have 3 monitors and the notifications show up on the farthest top right of the 3rd monitor. I thought it wasn't working at all, at first.

@dudik
Copy link
Owner

dudik commented Feb 1, 2022

At the moment it still does have to be added manually by applying this patch: #21.

@MostHated
Copy link

No worries, I was able to get it figured out not longer after I posted that. I kept trying to use diff filename 21.diff until I realized you had to actually use git to do the patch, lol. After that, I was good to go. 👍

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.

4 participants