Skip to content
This repository was archived by the owner on Jul 17, 2026. It is now read-only.

fix: 修复为我推荐页面点击MV直接展示评论区而不是mv本身的bug - #649

Merged
imsyy merged 4 commits into
SPlayer-Dev:devfrom
flystar233:dev
Dec 22, 2025
Merged

imsyy merged 4 commits into
SPlayer-Dev:devfrom
flystar233:dev

Conversation

@flystar233

Copy link
Copy Markdown
Contributor

No description provided.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Summary of Changes

Hello @flystar233, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

此拉取请求旨在解决一个用户体验问题,即在导航到MV页面时,用户可能会发现页面滚动到了底部,直接显示评论区而非MV内容。通过在路由导航完成后智能地重置主内容区域的滚动位置,确保了用户在访问新MV时总是从页面顶部开始浏览,从而提升了用户体验。

Highlights

  • Bug修复: 修复了点击MV时,页面错误地直接展示评论区而不是MV内容本身的bug。
  • 滚动位置重置: 在路由的 afterEach 钩子中增加了逻辑,当路由路径或查询参数 id 发生变化时,会自动将主内容区域的滚动位置重置到顶部。
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

你好,感谢你的贡献。这次的修改通过在 router.afterEach 中添加逻辑,解决了 MV 页面切换时滚动位置不重置的问题。这是一个很好的修复。我注意到目前的实现只针对了 id 查询参数的变化,这可能无法覆盖所有需要重置滚动条的场景(例如搜索页面)。我提供了一个建议,可以让这个逻辑更加通用和健壮。请查看我的具体评论。

Comment thread src/router/index.ts Outdated
Comment on lines +57 to +63
if (to.path !== from.path || to.query.id !== from.query.id) {
const mainContent = document.getElementById("main-content");
if (mainContent) {
const scrollContainer = mainContent.querySelector(".n-scrollbar-container");
if (scrollContainer) scrollContainer.scrollTop = 0;
}
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

目前的判断条件 to.path !== from.path || to.query.id !== from.query.id 只检查了路径和 id 查询参数的变化。这对于修复 MV 页面的 bug 是有效的,但对于其他使用不同查询参数的页面(例如使用 keyword 的搜索页)则无效。

为了使逻辑更通用,建议修改判断条件,以覆盖所有路径或查询参数变化的情况,同时忽略仅哈希值(hash)的变化。

一个更健壮的方法是比较 fullPath 中除去哈希值的部分。这样可以正确处理所有页面导航,使代码更具可维护性。

Suggested change
if (to.path !== from.path || to.query.id !== from.query.id) {
const mainContent = document.getElementById("main-content");
if (mainContent) {
const scrollContainer = mainContent.querySelector(".n-scrollbar-container");
if (scrollContainer) scrollContainer.scrollTop = 0;
}
}
if (to.fullPath.split("#")[0] !== from.fullPath.split("#")[0]) {
const mainContent = document.getElementById("main-content");
if (mainContent) {
const scrollContainer = mainContent.querySelector(".n-scrollbar-container");
if (scrollContainer) scrollContainer.scrollTop = 0;
}
}

@flystar233

Copy link
Copy Markdown
Contributor Author

update by gemini-code-assist

@flystar233 flystar233 changed the title fix: 修复点击MV直接展示评论区而不是mv本身的bug fix: 修复为我推荐页面点击MV直接展示评论区而不是mv本身的bug Dec 21, 2025
@imsyy
imsyy merged commit de03e8a into SPlayer-Dev:dev Dec 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants