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

Draft: Restore the missing login animation #2164

Closed
wants to merge 4 commits into from

Conversation

vanvugt
Copy link
Collaborator

@vanvugt vanvugt commented Mar 20, 2024

Restore the missing login animation, primarily for Ubuntu Dock which has disable-overview-on-startup.

https://bugs.launchpad.net/bugs/2058468

  • Use non-zero/undefined animation time.
  • Stop showing the dock before the animation starts.
  • Prevent the panel from showing before the animation starts (or don't animate the panel at all).
  • Disabling disable-overview-on-startup to re-enable the overview on startup has a broken animation.

Because gnome-shell stopped exporting the value in 45.0 it was being
treated as undefined, and therefore we had no startup animation.
Upstream gnome-shell has no parameter for this function, it was only
introduced in 8e4f4a8 to fix micheleg#1770 / LP#1965208. But those bugs have
resurfaced so we don't need it. And in fact removing the parameter will
help us to fix the same bugs, again.
So that dock.dash.opacity=0 and we don't see a dock before the startup
animation runs.
@vanvugt vanvugt changed the title docking: Hard code the startup animation time Restore the missing login animation Mar 20, 2024
@vanvugt vanvugt changed the title Restore the missing login animation Draft: Restore the missing login animation Mar 21, 2024
@vanvugt
Copy link
Collaborator Author

vanvugt commented Mar 21, 2024

I think I can do better and rewrite for the new startup animations in GNOME 45/46, rather than continuously adding on...

@vanvugt vanvugt closed this Mar 21, 2024
@vanvugt
Copy link
Collaborator Author

vanvugt commented Mar 21, 2024

Moved to #2166

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.

1 participant