Skip to content

feat: properly impl find for files#5741

Merged
IMB11 merged 12 commits intomodrinth:mainfrom
creeperkatze:properly-impl-search-for-files
Apr 16, 2026
Merged

feat: properly impl find for files#5741
IMB11 merged 12 commits intomodrinth:mainfrom
creeperkatze:properly-impl-search-for-files

Conversation

@creeperkatze
Copy link
Copy Markdown
Contributor

Adds a find and replace ui for files, using the internal functions from the ace editor.
Before, pressing Ctrl+F just showed the internal, ugly ui.

Example.mp4

Closes #5699

@modrinth-bot
Copy link
Copy Markdown
Member

modrinth-bot commented Apr 3, 2026

Pull request changelog

App

Added

Changed

  • The "Find and Replace" tool in the file editor for instance Files tab has been redesigned in the style of the rest of the platform. (Thank you @Creeperkatze)

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

  • The "Find and Replace" tool in the file editor for the Files tab has been redesigned in the style of the rest of the platform. (Thank you @Creeperkatze)

Deprecated

Removed

Fixed

Security

@creeperkatze creeperkatze changed the title feat(app): properly impl search for files feat(app): properly impl find for files Apr 3, 2026
@creeperkatze
Copy link
Copy Markdown
Contributor Author

Fixed lint

@creeperkatze creeperkatze changed the title feat(app): properly impl find for files feat: properly impl find for files Apr 3, 2026
@Prospector Prospector requested a review from IMB11 April 3, 2026 18:13
@IMB11
Copy link
Copy Markdown
Member

IMB11 commented Apr 13, 2026

Hi, this looks good, can you resolve the conflicts? Thanks

Copy link
Copy Markdown
Member

@IMB11 IMB11 left a comment

Choose a reason for hiding this comment

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

Just some things that need changing other than that looks good

Comment thread packages/ui/src/layouts/shared/files-tab/components/editor/EditorFindReplace.vue Outdated
Comment thread packages/ui/src/layouts/shared/files-tab/components/editor/FileEditor.vue Outdated
Comment thread packages/ui/src/layouts/shared/files-tab/components/FileNavbar.vue Outdated
@creeperkatze
Copy link
Copy Markdown
Contributor Author

Fixed!

@creeperkatze
Copy link
Copy Markdown
Contributor Author

Slight adjustment, the results label is now oriented left
image

@creeperkatze creeperkatze requested a review from IMB11 April 15, 2026 20:21
@IMB11 IMB11 enabled auto-merge April 16, 2026 10:59
@IMB11 IMB11 added this pull request to the merge queue Apr 16, 2026
Merged via the queue into modrinth:main with commit 5b5c8c0 Apr 16, 2026
6 checks passed
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.

[Feature Request] Search items in a file

3 participants