Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve N3TexViewer usability: show file path and disable auto-resizing on navigation. #259

Merged
merged 2 commits into from
Nov 13, 2024

Conversation

stevewgr
Copy link
Member

Description

This pull request enhances the usability of N3TexViewer by displaying the absolute file path of the currently opened texture in the status bar and by disabling automatic window resizing when navigating between textures. These changes improve both visibility and consistency, providing a smoother experience when working with textures.

Details

  • Display Absolute File Path: The status bar now shows the full path of the currently opened texture, making it easier to identify the loaded file’s location without external references.

  • Disable AdjustWindowSize on Navigation: Previously, the window resized automatically when navigating textures with PageDown and PageUp shortcuts. This behavior often disrupted the user experience, especially with small textures (e.g., 32x32), as it shrank the window to an inconvenient size. Now, the window only adjusts size for the initial texture load, remaining consistent during navigation.

When opening the initial texture, the window automatically adjusts size
appropriately. However, this resizing also occurred when navigating
through textures with PageDown and PageUp shortcuts, causing the editor
window to resize with each new texture.

This behavior was particularly disruptive for very small textures (e.g.,
32x32), as it would shrink the window to an unviewable size. Disabling
AdjustWindowSize for navigation prevents this issue, allowing for a
more consistent and comfortable browsing experience.
Shows the full path of the currently opened texture in the status bar
to improve clarity on the loaded file.
@stevewgr stevewgr requested review from a team November 13, 2024 03:04
@stevewgr stevewgr self-assigned this Nov 13, 2024
Copy link
Contributor

@xGuTeK xGuTeK left a comment

Choose a reason for hiding this comment

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

Looks fine :)

@stevewgr
Copy link
Member Author

Build passing: https://github.com/ko4life-net/ko/actions/runs/11809860391

Merging.

@stevewgr stevewgr merged commit 2b24dea into master Nov 13, 2024
2 checks passed
@stevewgr stevewgr deleted the n3dxt-improve branch November 13, 2024 04:19
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.

2 participants