Skip to content

Try animating width/height to a container div -- alternative to AnimatedContainer#5

Merged
alexspeller merged 1 commit intomainfrom
lukemelia/cs-283-experiment-with-not-needing
Mar 2, 2021
Merged

Try animating width/height to a container div -- alternative to AnimatedContainer#5
alexspeller merged 1 commit intomainfrom
lukemelia/cs-283-experiment-with-not-needing

Conversation

@lukemelia
Copy link
Copy Markdown
Contributor

No description provided.

@linear
Copy link
Copy Markdown

linear bot commented Mar 1, 2021

CS-283 Experiment with not needing AnimatedContainer

Add {{sprite}} to div you want to animate, and animate it.

@lukemelia lukemelia added the enhancement New feature or request label Mar 1, 2021
@alexspeller alexspeller merged commit 4dd5827 into main Mar 2, 2021
@alexspeller alexspeller deleted the lukemelia/cs-283-experiment-with-not-needing branch March 2, 2021 01:47
ef4 added a commit that referenced this pull request Nov 18, 2022
moved file tree into it's own component & reflecting loaded file in query params
habdelra added a commit that referenced this pull request Apr 10, 2026
- Move statusIcon getter from TestResultEntry.embedded to FieldDef class
  to eliminate CSS pseudo-element duplication (Copilot comment #6)
- Move isComplete getter from TestModuleResult.embedded to FieldDef class
  and add totalCount getter for null-safe access
- Show "running..." for modules with pending tests instead of misleading
  green "0/N passed" (codex-connector comment, Copilot comment #2)
- Show explicit fail count ("X passed, Y failed") when failures exist
  instead of just "X/Y passed" in red (Copilot comment #3)
- Add role="img" and aria-label to status icon spans for screen reader
  accessibility (Copilot comment #5)
- Keep full red border on failed modules per reviewer preference

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants