-
Notifications
You must be signed in to change notification settings - Fork 162
trigger cosmic-osd display identifier OSD on each screen to ease display arrangement configurations for users #1554
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
base: master
Are you sure you want to change the base?
trigger cosmic-osd display identifier OSD on each screen to ease display arrangement configurations for users #1554
Conversation
|
Thanks for working on it. There was a previous discussion about it and a mockup. The idea is to always show the numbers when there are 2+ displays connected. We've seen cases where people struggled to identify which display is which while trying to arrange them so having the numbers always present while Displays page in Settings is focused would be helpful. You can see the placement and the styling of the OSD on the mockup. Accent color background is used to make the OSD more immediately apparent. |
55506d0 to
5629f00
Compare
|
I think that the way this is implemented is mostly fine, but there should probably be some sort of timeout for the osd, so that if settings fails to dismiss it properly, it disappears on its own after a few seconds. For example, when testing if, if I navigate to the displays page and close the settings app, the osd does not disappear unless I navigate away from the page. @pop-os/ux Do you think it could appear on first entering the page, then fade away after a few seconds, and reappear when a display is hovered? |
I was wondering about that but we've discussed it with @WatchMkr and thought it best to have the numbers show as long as the Displays page is focused. |
This should only happen, if you forcefully kill the cosmic-settings process. When you gracefully close the settings app via the "x" button, then the OSD should disappear as well. Nonetheless, during the development I have implemented two different proof of concepts to deal with a cosmic-settings app crash/kill and still be able to hide the display identifier OSD.
Option 1 feels a bit hacky, option 2 feels like unnecessary message spam on the d-bus (but manageable, since its a 1-to-n connection from sender to subscriber). I did not feel comfortable with either option so I did not included them. Feel free to let me know what you think would be the best option.
I can implement a fade out animation as well. The more specific you are with those details, timings and so on, the better. |
|
Here is a video of what the current develop state in this pull request looks like: 2025-11-12.21-17-46.mp4 |
This sounds fine to me for now. Maybe even with a 1 second timer.
It happened when I closed settings using the |
|
Alright, then I will do it that way. |
…y arrangement configurations for users
… as a small square at the top of each monitor with the accent color
…se OSDs when leaving the display page, closing the settings app gracefully or forcefully with super key + q
4500339 to
8b10f95
Compare
|
addressed the feedback |
jacobgkau
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to stop working after a display is connected or disconnected. E.g. it works on a laptop (oryp8) with just its internal screen, but no longer shows (even after restarting COSMIC Settings) if I plug in another display. If I boot with two displays, then Settings first shows the OSDs on both, but it no longer shows the OSD at all if I unplug the second display.
I think the OSD's also showing up a little larger than it does in the mockup. Both the current implementation and the mockup had the OSD a little smaller than the date/time in the panel, but the date/time in the mockup's much shorter since it was an older version with an abbreviated month and no "at" word. (In the mockup, the OSD also appears to be smaller than the word "Displays" in Settings, whereas it's larger than that word in the current implementation; just another reference for size.) It doesn't have to be exact, but it might be good to make it a little smaller to better match the mockup for now.
|
@jacobgkau oh wow, good catch! I haven't thought about this scenario with the display hot-plugging. I have pushed fixes for that and the visual changes to the OSD pull request. |
implements #885
requires pop-os/cosmic-osd#151
preview:
2025-11-09.21-51-08.mp4