Skip to content

Add project start screen#45

Merged
MikeSzklarz merged 4 commits into
lanl:stagingfrom
alex-ur88:feature/project-start-screen
May 21, 2026
Merged

Add project start screen#45
MikeSzklarz merged 4 commits into
lanl:stagingfrom
alex-ur88:feature/project-start-screen

Conversation

@alex-ur88
Copy link
Copy Markdown
Contributor

@alex-ur88 alex-ur88 commented May 20, 2026

Summary

Adds a project start screen for the empty/no-images-loaded state so users have clear first actions when launching the tool.

Changes

  • Adds a centered start screen with Open Project and Open Image Folder actions
  • Shows recent project and recent image folder shortcuts after those paths have been used
  • Stores recent shortcuts locally with QSettings
  • Hides the start screen once an image dataset is loaded and restores it when the dataset is cleared
  • Applies Ruff cleanup needed because CI runs pre-commit across all files
  • Stabilizes viewport action toggle tests that were already failing on clean upstream/staging

Notes

  • This PR only covers the project start screen slice from the earlier combined workflow PR
  • Recent shortcuts are local app convenience state, not .annoproj project state
  • The viewport test fix is included because the same failures reproduced on clean upstream/staging, not because the start screen introduced them

Testing

  • python -m pytest src/tests/integration/test_viewport_actions.py -q
  • python -m ruff check src/tests/integration/test_viewport_actions.py
  • python -m ruff format --check src/tests/integration/test_viewport_actions.py
  • CI checks passed

@alex-ur88 alex-ur88 force-pushed the feature/project-start-screen branch from bbd09ae to e36e3a0 Compare May 20, 2026 06:35
@MikeSzklarz
Copy link
Copy Markdown
Collaborator

If you could just comment here what files are changed.

For the future commits dont do the ruff pre commit even if it fails. We will remove the ruff stuff as its causing issues for checking whats changed in PR's. For example it says 17 files changes even though it was probably only a couple.

You dont need to change anything now, just lmk which files were changed for the actual implementation

@alex-ur88
Copy link
Copy Markdown
Contributor Author

alex-ur88 commented May 21, 2026

If you could just comment here what files are changed.

For the future commits dont do the ruff pre commit even if it fails. We will remove the ruff stuff as its causing issues for checking whats changed in PR's. For example it says 17 files changes even though it was probably only a couple.

You dont need to change anything now, just lmk which files were changed for the actual implementation

Yeah absolutely. The actual project start screen implementation changed the following files:

  • src/views/annomate/window.py
  • src/views/app_window.py

Separately, I also changed 'src/tests/integration/test_viewport_actions.py' for the small test stabilization fix around the image-viewer grid/center-crop toggle tests. The remaining changed files are from the Ruff/pre-commit cleanup stuff.

Understood on the Ruff/pre-commit point. For future PRs, I won’t apply Ruff cleanup just to get the check passing from now on.

@MikeSzklarz MikeSzklarz merged commit 486de9a into lanl:staging May 21, 2026
4 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.

2 participants