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

Migrate FtProgressBar to the composition API #6041

Merged

Conversation

absidue
Copy link
Member

@absidue absidue commented Oct 30, 2024

Migrate FtProgressBar to the composition API

Pull Request Type

  • Bugfix
  • Refactoring - Composition API migration

Description

This pull request migrates the FtProgressBar component to the composition API. It also removes the slow animation and bumps the z-index to 4 so that it is displayed in front of the side bar instead of behind it.

Testing

Refresh your subscriptions and check that you can see the progress bar at the bottom of the window.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 741268a

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) October 30, 2024 14:03
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 30, 2024
Copy link
Collaborator

@kommunarr kommunarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested web & simulated mobile.

@FreeTubeBot FreeTubeBot merged commit d678863 into FreeTubeApp:development Oct 30, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 30, 2024
@absidue absidue deleted the progress-bar-composition-api branch October 30, 2024 14:22
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.

5 participants