Skip to content

Conversation

9romise
Copy link
Contributor

@9romise 9romise commented Oct 10, 2025

Prerequisites checklist

What is the purpose of this pull request?

What changes did you make? (Give an overview)

Fix the issue where navigating from FileGroupItem to the configs view did not open or scroll to the correct ConfigItem.

The scrolling problem occurred because el.$el from a Vue ref points to the first node of the ConfigItem component, which is a vue-ignore comment node, causing scrollIntoView to fail.

Related Issues

Is there anything you'd like reviewers to focus on?

@eslint-github-bot
Copy link

Hi @9romise!, thanks for the Pull Request

The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.

  • The commit message tag wasn't recognized. Did you mean "docs", "fix", or "feat"?
  • There should be a space following the initial tag and colon, for example 'feat: Message'.
  • The length of the commit message must be less than or equal to 72

To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page.

Read more about contributing to ESLint here

@9romise 9romise changed the title fix(configs): ensure correct config item is opened and scrolled into view from FileGroupItem fix: ensure correct config item is opened and scrolled into view from FileGroupItem Oct 10, 2025
@eslint-github-bot
Copy link

Hi @9romise!, thanks for the Pull Request

The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.

  • The length of the commit message must be less than or equal to 72

To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page.

Read more about contributing to ESLint here

@9romise 9romise changed the title fix: ensure correct config item is opened and scrolled into view from FileGroupItem fix: scroll to correct ConfigItem when navigating from FileGroupItem Oct 10, 2025
@eslint-github-bot eslint-github-bot bot added the bug Something isn't working label Oct 10, 2025
@9romise 9romise marked this pull request as ready for review October 10, 2025 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant