Skip to content

fix: Implement startupPreparedHandler for overview control#2493

Open
mbevc1 wants to merge 6 commits into
home-sweet-gnome:masterfrom
mbevc1:patch-1
Open

fix: Implement startupPreparedHandler for overview control#2493
mbevc1 wants to merge 6 commits into
home-sweet-gnome:masterfrom
mbevc1:patch-1

Conversation

@mbevc1

@mbevc1 mbevc1 commented May 1, 2026

Copy link
Copy Markdown

Added a startupPreparedHandler to manage overview visibility during startup.

Fixes: #2492

Added a startupPreparedHandler to manage overview visibility during startup.

fixes: home-sweet-gnome#2492
@mbevc1

mbevc1 commented May 5, 2026

Copy link
Copy Markdown
Author

@charlesg99 I'd appreciate if someone could please review this 🙏

@charlesg99

Copy link
Copy Markdown
Member

Hey Marko, I tested your branch, but on my end it breaks my system on log out/in, not sure what's causing it. Also, maybe I don't get the desired result; after logging in I still see the overview and instead of getting the animation going out of the overview, it goes out of overview like if there was a very laggy animation. Are you seing this as well?

Ideally we would find a way to stay out of the overview on startup, but on my system at least, when the extension init code executes, the animation going to the overview is already running. Did you test it in a vm? I'm asking because I get a different timing if I try it out using a nested wayland session. Thanks!

mbevc1 added 5 commits May 5, 2026 18:52
Removed commented-out code related to startup animation handling.
Refactor startup animation handling to ensure proper ordering and restore original function if necessary.
Refactor startup overview handling to ensure correct behavior during GNOME Shell startup.
@mbevc1

mbevc1 commented May 5, 2026

Copy link
Copy Markdown
Author

Hey, yeah that is VM I'm testing on. you're right there were some inconsistencies and I've pushed new changes which seem to work as expected now. I'm using Fedora 44 on Gnome 50.

@mbevc1

mbevc1 commented May 6, 2026

Copy link
Copy Markdown
Author

@charlesg99 is this change working for you now? 🤔

@mbevc1

mbevc1 commented May 12, 2026

Copy link
Copy Markdown
Author

Mind having another review please?

@Nilvus

Nilvus commented Jun 5, 2026

Copy link
Copy Markdown

Hi @mbevc1,
Your PR solve the issue on my side. I just see overview 1 second before seeing it disappear.

@mbevc1

mbevc1 commented Jun 9, 2026

Copy link
Copy Markdown
Author

@charlesg99 did you managed to re-check this?

@mbevc1

mbevc1 commented Jun 25, 2026

Copy link
Copy Markdown
Author

@charlesg99 any updates on this?

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.

[Gnome 50] Disable show overview on startup not working

3 participants