Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

[Backport 5.1] Fuzzy finder: allow line number at the end of a filename#55252

Merged
BolajiOlajide merged 1 commit into
5.1from
backport-55064-to-5.1
Jul 25, 2023
Merged

[Backport 5.1] Fuzzy finder: allow line number at the end of a filename#55252
BolajiOlajide merged 1 commit into
5.1from
backport-55064-to-5.1

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Previously, it was not possible to navigate straight to a specific line number with the "Files" tab of the fuzzy finder. This commit adds the ability to specify a line number with a :NUMBER suffix. This feature makes it possible to copy-paste error messages that use the FILE:LINE:COLUMN format and quickly open the location with the fuzzy finder.

Test plan

Previously, it was not possible to navigate straight to a specific line
number with the "Files" tab of the fuzzy finder. This commit adds the
ability to specify a line number with a `:NUMBER` suffix. This feature
makes it possible to copy-paste error messages that use the
`FILE:LINE:COLUMN` format and quickly open the location with the fuzzy
finder.

## Test plan

- New storybook test
- New unit tests for query parsing
<!-- All pull requests REQUIRE a test plan:
https://docs.sourcegraph.com/dev/background-information/testing_principles
-->

(cherry picked from commit 0633c1d)
@sourcegraph-bot

Copy link
Copy Markdown
Contributor

📖 Storybook live preview

@olafurpg

Copy link
Copy Markdown
Contributor

Opened PR https://github.com/sourcegraph/sourcegraph/pull/55257 to document this backport.

@olafurpg

Copy link
Copy Markdown
Contributor

I manually built this branch locally to confirm that the behavior works as expected.

@olafurpg

Copy link
Copy Markdown
Contributor

I approved the UI changes in chromatic.

@olafurpg olafurpg requested a review from BolajiOlajide July 25, 2023 11:03
@BolajiOlajide BolajiOlajide merged commit 65b0a23 into 5.1 Jul 25, 2023
@BolajiOlajide BolajiOlajide deleted the backport-55064-to-5.1 branch July 25, 2023 11:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backports cla-signed team/code-search Issues owned by the code search team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants