docs: improve README product framing and visuals#44
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 56b8121f48
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ```bash | ||
| cd services/racing-analytics | ||
| python -m venv .venv | ||
| . .venv/Scripts/activate |
There was a problem hiding this comment.
This block is marked as bash, but after python -m venv .venv on macOS/Linux the activation script is .venv/bin/activate; .venv/Scripts/activate is the Windows layout. As written, Unix users copying the root README hit No such file or directory before installing deps or running FastAPI, so include both platform paths or switch this command to . .venv/bin/activate.
Useful? React with 👍 / 👎.
| .beat { animation: beat 1.8s ease-in-out infinite; transform-origin: center; transform-box: fill-box; } | ||
| @keyframes beat { 0%, 100% { transform: scale(1); opacity: 0.75; } 50% { transform: scale(1.08); opacity: 1; } } | ||
| @media (prefers-reduced-motion: reduce) { | ||
| .beat, animateMotion { animation: none !important; } |
There was a problem hiding this comment.
For users with prefers-reduced-motion: reduce, this rule still leaves the provider dots moving because animation: none only disables CSS animations, while the seven <animateMotion repeatCount="indefinite"> elements below are SMIL animations. Since this SVG is embedded in the README as an animated preview, provide a static fallback or remove/disable the animateMotion elements under the reduced-motion path so the preference is actually respected.
Useful? React with 👍 / 👎.
| <rect class="panel" x="0" y="0" width="500" height="280" rx="24"/> | ||
| <rect x="22" y="20" width="456" height="42" rx="18" fill="rgba(255,255,255,0.06)"/> | ||
| <text class="tiny" x="42" y="47">provider pulse</text> | ||
| <g class="ticker" transform="translate(175 28)"> |
There was a problem hiding this comment.
Because .ticker animates the CSS transform property on this same <g>, browsers treat that property as overriding the SVG transform="translate(175 28)" attribute while the animation runs. In the README preview this makes the odds ticker start at the panel origin and slide over the provider pulse label instead of staying in the header lane; move the static offset to a parent group or include it in the keyframes.
Useful? React with 👍 / 👎.
1 participant
Summary
Improves the Bettin2Win README so the project is easier to understand as a product and portfolio piece.
Changes
Verification