Skip to content

Minor performance improvements to the Hashtag view#5927

Merged
FreeTubeBot merged 1 commit intoFreeTubeApp:developmentfrom
absidue:hashtag-perf
Oct 24, 2024
Merged

Minor performance improvements to the Hashtag view#5927
FreeTubeBot merged 1 commit intoFreeTubeApp:developmentfrom
absidue:hashtag-perf

Conversation

@absidue
Copy link
Member

@absidue absidue commented Oct 23, 2024

Minor performance improvements to the Hashtag view

Pull Request Type

  • Performance improvement

Related issue

#5825 (comment)

Description

This pull request implements two small performance improvements to the hashtag view. The first one is that we don't have to call resetData() in the mounted hook, as at the time that the mounted hook is called all the fields still have their default values. The second one is to only watch the hashtag param on the current route object, as that is the only property that could change while you are on the hashtag page.

Testing

  1. Paste https://www.youtube.com/hashtag/shorts into the search bar and hit enter
  2. It should show the hashtag page for #shorts
  3. Paste https://www.youtube.com/hashtag/asmr into the search bar and hit enter
  4. It should switch to the hashtag page for #asmr

Desktop

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

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) October 23, 2024 21:20
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 23, 2024
@FreeTubeBot FreeTubeBot merged commit f6e7344 into FreeTubeApp:development Oct 24, 2024
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 24, 2024
@absidue absidue deleted the hashtag-perf branch October 24, 2024 13:08
Soham456 pushed a commit to Soham456/FreeTube that referenced this pull request Dec 5, 2024
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