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

Don't show Ringing window fullscreen and center it rather #1876

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sidevesh
Copy link
Contributor

@sidevesh sidevesh commented Oct 13, 2024

Made this change because when that dialog is maximized it was broken and only covered top right quadrant for some reason looking very out of place.
Also removed the dismissal of dialog on hover over it because it seemed abrupt and made it so the main button the dialog can't be clicked at all, making it a very confusing UX that seemed broken and as if its crashing abruptly.

@ferdnyc
Copy link
Member

ferdnyc commented Oct 23, 2024

There is so much wrong with that window, in every conceivable way.

As I mentioned in #1875 (comment) it might even be worth considering that we just dump the window entirely in favor of a notification. Now that action buttons on notifications persist into the drawer and can still be clicked even if you miss the popup, that's not so unworkable as it used to be.

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 this pull request may close these issues.

2 participants