Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR bumps sview to v0.2.1 and focuses on UI polish: adding a packaged app icon (used for the window and About dialog), improving hidden-file controls across the tree and browser views, and refining selection/focus behavior for smoother navigation.
Changes:
- Add a packaged application icon and wire it into app/window + About dialog UI.
- Add “Show Hidden Files” state (persisted) and update tree + browser filtering to respect it.
- Improve navigation UX (auto-select first item; focus restoration; directory-first sorting in the table).
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
sview/qt/tree.py |
Adds hidden-directory filtering and custom key navigation for the directory tree. |
sview/qt/table.py |
Adds directory-first sorting via a custom table item and resets/auto-selects after refresh. |
sview/qt/icon_view.py |
Auto-focuses the first item after loading items. |
sview/qt/main_window.py |
Adds toolbar menu button, hidden-files toggle + persistence, focus restoration, and enhanced About dialog with icon. |
sview/qt/app.py |
Sets the application/window icon from the packaged PNG. |
sview/qt/sview_icon.png |
Adds the application icon asset to the repo/package. |
sview/__init__.py |
Bumps __version__ to 0.2.1. |
pyproject.toml |
Bumps project version and includes the icon in package data. |
CHANGELOG.md |
Adds a 0.2.1 changelog entry. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the application to version 0.2.1 and introduces several user interface enhancements, including a new application icon, improved About dialog, and significant improvements to hidden-file controls and tree navigation. It also adds a toolbar menu, preferences stub, and polish to the browser's selection and focus behaviors.
UI/UX Improvements:
sview_icon.png), set it as the window icon, and displayed it in the About dialog for a more polished appearance. [1] [2] [3] [4] [5]Hidden Files and Tree Navigation:
Other Notable Changes:
These changes collectively enhance the application's usability, visual polish, and user control over file visibility and navigation.