Bug Fixes and UX Improvements

[PierreGac]

Version 1.0.1 - Bug Fixes and UX Improvements

Release Date: [Current Date]

This release focuses on critical bug fixes, improved user experience, and enhanced notification management. All changes maintain backward compatibility with v1.0.0.

🐛 Bug Fixes

Notification System

• Fixed notification icons persisting after viewing model details
  • Notification badges now disappear immediately when model details are opened
  • Window title updates automatically without requiring manual refresh
  • Added NotificationStateManager to track read/unread state for notes and updates
  • Implemented automatic UI refresh when notifications are marked as read

Model Submission

• Fixed RelativePath including Materials folder when submitting from Project view

  • Improved GetDefaultRelativePath() to detect model root folder correctly
  • Automatically walks up directory tree to find folder containing FBX/OBJ files
  • Excludes common subfolders (Materials, Textures, Prefabs, Scripts, Animations)
  • Prioritizes mesh files (FBX/OBJ) in selection for accurate path detection

• Fixed version deletion errors preventing access to model details

  • Refreshes version list after deletion to get updated available versions
  • Improved error handling when opening next version after deletion
  • Handles edge case when last version is deleted gracefully
  • Better user feedback with informative error messages

First Run Wizard

• Fixed configuration detection issues
  • Improved IsConfigured() to properly check for default values
  • Fixed detection of "anonymous" username as unconfigured state
  • Added validation in SaveConfiguration() with user feedback
  • Enhanced error handling and logging for configuration failures
  • Window now properly focuses when opened

Image Addition

• Fixed image addition not working correctly
  • Improved path normalization for drag-and-drop operations
  • Fixed Texture2D asset drag-and-drop path conversion
  • Added case-insensitive duplicate detection
  • Better error messages for invalid files
  • Added user feedback for successful additions

✨ Improvements

Validation & User Feedback

• Implemented comprehensive changelog validation

  • Real-time validation feedback in submission form
  • Uses ChangelogValidator for comprehensive checks
  • Shows validation errors and suggestions
  • Better guidance for update mode changelog requirements

• Enhanced duplicate detection

  • Improved null checks and whitespace trimming
  • Better error messages for duplicate model/version detection
  • Validates semantic version ordering for updates
  • More informative validation feedback

User Interface

• Added loading overlay during refresh operations

  • Full-screen overlay blocks UI interaction during loading
  • Shows progress indicator with animated dots
  • Prevents accidental clicks during refresh
  • Clear loading messages for different operations

• Improved update icon visibility

  • Update badges now appear in all view modes (List, Grid, Image-only)
  • Respects read state for update notifications
  • Consistent badge display across all views
  • Update count in window title reflects unread updates only

Code Quality

• Standardized error logging

  • Replaced Debug.Log* calls with centralized ErrorLogger
  • Added error categorization using ErrorHandler
  • Improved error context and debugging information
  • Consistent error handling patterns

• Performance optimizations

  • Cached texture extensions array to avoid allocations
  • Cached mesh names array to reduce LINQ allocations
  • Used StringBuilder for tooltip construction
  • Replaced string.Concat with string interpolation

🔧 Technical Changes

New Utilities

• NotificationStateManager (Assets/ModelLibrary/Editor/Infrastructure/Utils/NotificationStateManager.cs)
  • Manages notification read/unread state using EditorPrefs
  • Tracks notes and update notifications separately
  • Provides methods to mark notifications as read
  • Enables automatic UI updates when state changes

API Changes

• Public Methods Added
  • ModelLibraryWindow.UpdateNotesCount() - Recalculates unread notes count
  • ModelLibraryWindow.UpdateUpdateCount() - Recalculates unread updates count
  • ModelLibraryWindow.UpdateWindowTitle() - Refreshes window title with current counts

Internal Improvements

• Enhanced path validation and normalization
• Improved async error handling
• Better null safety checks throughout
• More comprehensive XML documentation

📋 Migration Notes

No migration required. This release is fully backward compatible with v1.0.0.

Breaking Changes

None

Deprecations

None

🧪 Testing

All fixes have been validated with:

• Manual testing of notification persistence
• Path detection with various folder structures
• Version deletion edge cases
• Image addition with multiple formats
• Loading overlay during refresh operations

📝 Known Issues

• Metallic/Specular rendering in 3D Preview window (investigation ongoing)
• Background notification checking without browser launch (planned for future release)
• Version revert functionality (planned for future release)

🙏 Acknowledgments

Thanks to all users who reported issues and provided feedback that led to these improvements.

---

Full Changelog: See commit history for detailed changes

---

This discussion was created from the release Bug Fixes and UX Improvements.