Skip to content

Conversation

@Traben-0
Copy link
Contributor

Description

refactor SourceSets and the list for the PreprocessorJumpAction:

  • corrects list ordering
  • support override files from versions/<version>/src/
  • remove redundant full path from the list string
  • layout list cells into nicer columns
  • add simple "key" to streamline keyboard searching further
  • add indicators for main & override files

also:

  • upon filtering the list via search, it auto selects the first result for keyboard navigation streamlining
  • focused the search field upon opening for keyboard navigation streamlining
  • allows double clicking a list entry to open it immediately

Related Issue(s)

Fixes #19
partially fixes #18

How to test

use the preprocessor file jump dialogue box

image image

Release Notes

- Added support for override files within `versions/<version>/src/` to the "jump to preprocessed files" action.
- Improved the visual clarity of the list in the "jump to preprocessed files" dialogue window.
- Improved keyboard navigation of the "jump to preprocessed files" dialogue window.
- Allowed double clicking on entries in the "jump to preprocessed files" dialogue window to open the file.
- Fixed sorting of entries in the "jump to preprocessed files" dialogue window.

- correct list ordering
- support override files from `versions/<version>/src/`
- remove redundant full path from list string
- layout list cells into nicer columns
- add simple "key" to streamline keyboard searching further
- add indicators for main & override files
- first half `toRelativePath()` related logic
- second half `toString()` related logic

also removes unneeded diff by restoring `toRelativePath()` as the final path getter
@Traben-0
Copy link
Contributor Author

This wasn't particularly made for those issues, one isnt fully addressed

I've just been modifying this for personal use and figured i'd see if the changes were wanted here

@Wyvest Wyvest merged commit 11cfdd8 into Polyfrost:main Sep 25, 2025
2 of 3 checks passed
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.

Jump-to-file does not support per-version overrides Improve jump-to-file ergonomics

2 participants