Skip to content

Conversation

@interim17
Copy link
Contributor

DRAFT: waiting on some final color feedback from UX, but this the basic feature

Problem

Closes #136 and #137

Solution

Conditionally render overlay, spinner and text based on state of the app.
Screenshot 2025-11-12 at 10 57 29 AM
Screenshot 2025-11-12 at 10 57 11 AM
Screenshot 2025-11-12 at 10 56 55 AM

interim17 and others added 19 commits November 10, 2025 11:06
@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 20.19% 367 / 1817
🔵 Statements 20.19% 367 / 1817
🔵 Functions 40% 24 / 60
🔵 Branches 71.05% 81 / 114
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/components/Viewer/index.tsx 0% 0% 0% 0% 1-4, 6-11, 13-20, 22-23, 25-37, 39, 41, 43
src/state/store.ts 0% 0% 0% 0% 1, 44, 46-52, 54-56, 58, 60-63, 65-76, 78-79, 81-82, 84-86, 89-90, 93, 96-100, 102-104, 106-108, 110-113, 115-123, 125-136, 138-149, 151-153, 155, 157-162, 164-173, 176-179, 182-188, 191-194, 196-197, 199-205, 207-220, 222-236, 238-239, 243-249, 251-254, 256-261, 263-267, 269-273, 275, 277-278, 282-284, 286-289, 291-294, 296-299, 301-304, 306-317, 319-322, 325-340
src/style/themeRoot.tsx 0% 0% 0% 0% 1, 5-9, 11-17, 19-24, 26
Generated in workflow #149

@github-actions
Copy link

PR Preview Action v1.6.2

🚀 View preview at
https://AllenCell.github.io/cellpack-client/pr-preview/pr-154/

Built to branch gh-pages at 2025-11-12 18:59 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Base automatically changed from fix/prefetch to main November 13, 2025 20:04
@meganrm
Copy link
Contributor

meganrm commented Nov 15, 2025

@interim17 what's the status with this? I know you said you needed some feedback from @thao-do

@interim17
Copy link
Contributor Author

@interim17 what's the status with this? I know you said you needed some feedback from @thao-do

I think we are good to go? But I made a new PR because this branch got stale:
#162

@interim17 interim17 closed this Nov 19, 2025
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.

Loading screen

3 participants