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

Hide Progress bar on turbo:load #1036

Merged
merged 1 commit into from
Nov 17, 2023
Merged

Conversation

seanpdoyle
Copy link
Contributor

Closes #962

Prior to this commit, the progress bar is hidden between the Turbo Drive visits' turbo:before-fetch-response event and the turbo:render event.

This commit changes that behavior to hide the bar when the Visit is complete (some time prior to the turbo:load event).

@seanpdoyle seanpdoyle force-pushed the issue-962 branch 5 times, most recently from 3791846 to 96515ff Compare October 12, 2023 20:53
Closes [hotwired#962][]

Prior to this commit, the progress bar is hidden between the Turbo Drive
visits' `turbo:before-fetch-response` event and the `turbo:render`
event.

This commit changes that behavior to hide the bar when the Visit is
complete (some time prior to the `turbo:load` event).

[hotwired#962]: hotwired#962
@afcapel afcapel merged commit cc263b4 into hotwired:main Nov 17, 2023
@afcapel
Copy link
Collaborator

afcapel commented Nov 17, 2023

Thanks Sean!

@seanpdoyle seanpdoyle deleted the issue-962 branch November 17, 2023 16:11
domchristie pushed a commit to domchristie/turbo that referenced this pull request Jul 20, 2024
Closes [hotwired#962][]

Prior to this commit, the progress bar is hidden between the Turbo Drive
visits' `turbo:before-fetch-response` event and the `turbo:render`
event.

This commit changes that behavior to hide the bar when the Visit is
complete (some time prior to the `turbo:load` event).

[hotwired#962]: hotwired#962
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Make progress bar finish on turbo:load
2 participants