Skip to content

add refresh button to stride ui#55

Merged
elainethale merged 11 commits intomainfrom
refresh_recent_projects
Apr 8, 2026
Merged

add refresh button to stride ui#55
elainethale merged 11 commits intomainfrom
refresh_recent_projects

Conversation

@irmatgit
Copy link
Copy Markdown
Contributor

@irmatgit irmatgit commented Mar 6, 2026

These changes were required to put stride package in a UI.

  • Added a refresh recent project list button in the create_app_no_project app.
  • Synchronize size of the config.json with the get_recent_project()
  • Adjusted grey color level in the UI to make the text more readable
  • Define MAX_CACHED_PROJECTS variable to limit memory usage in small containers

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 86.45038% with 71 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.55%. Comparing base (6f2b52a) to head (a1ad20c).
⚠️ Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
src/stride/ui/app.py 47.82% 36 Missing ⚠️
src/stride/ui/settings/callbacks.py 15.00% 17 Missing ⚠️
src/stride/ui/settings/layout.py 8.33% 11 Missing ⚠️
src/stride/cli/stride.py 33.33% 4 Missing ⚠️
tests/test_app_cache.py 99.25% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
- Coverage   79.53%   75.55%   -3.99%     
==========================================
  Files          47       50       +3     
  Lines        6855     7935    +1080     
==========================================
+ Hits         5452     5995     +543     
- Misses       1403     1940     +537     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@irmatgit irmatgit requested a review from daniel-thom March 6, 2026 12:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the STRIDE UI to better support packaging into a UI-centric workflow by improving project switching UX, limiting resource usage from concurrently open projects, and adjusting dark theme text colors.

Changes:

  • Added a refresh button next to the project switcher dropdown (both with and without a loaded project).
  • Introduced an LRU-style eviction policy for cached/open projects with a MAX_CACHED_PROJECTS limit.
  • Increased the default recent-projects list size to 10 and adjusted dark theme text color variables.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
src/stride/ui/project_manager.py Updates the default number of recent projects returned to 10.
src/stride/ui/assets/dark-theme.css Tweaks dark theme text color variables.
src/stride/ui/app.py Adds refresh UI + callbacks for project dropdown, and implements capped project caching with eviction.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

irmatgit and others added 3 commits April 8, 2026 08:44
@elainethale elainethale merged commit 5b4fb74 into main Apr 8, 2026
4 of 5 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 8, 2026
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.

3 participants