docs(inductor): correct currentRating to maxCurrentRating in specs !! - #819
docs(inductor): correct currentRating to maxCurrentRating in specs !!#819KrishnaX12 wants to merge 3 commits into
Conversation
…ations and standardize micro symbols
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| name="L1" | ||
| footprint="0603" | ||
| inductance="10μH" | ||
| inductance="10µH" |
There was a problem hiding this comment.
What did you change in here?
There was a problem hiding this comment.
@imrishabh18 While they look visually identical, the underlying Unicode characters are actually different:
- Old:
U+03BC(Greek Small Letter Mu,μ) - New:
U+00B5(Micro Sign,µ)
Currently,@tscircuit/format-si-unitonly recognizesU+00B5. Because the docs usedU+03BC, the live<CircuitPreview>failed to recognize the prefix and rendered10Hinstead of10µH.
I also opened fix: add support for Greek small letter mu (U+03BC) alongside micro sign (U+00B5) in unit parsing format-si-unit#18 to fix this bug upstream across the entire framework, and corrected an incorrect specification property name (currentRating→maxCurrentRating).
KrishnaX12
changed the title
|
This PR has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs. |
|
This PR was closed because it has been inactive for 1 day since being marked as stale. |
|
conflicts |
|
This PR has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs. |
KrishnaX12
changed the title
|
This PR has been automatically marked as stale because it has had no recent activity. It will be closed if no further activity occurs. |
KrishnaX12
changed the title
4 participants
Description
Specification Correction: Under Specifications, corrected the bullet point from
- **currentRating**to- **maxCurrentRating**, aligning it with@tscircuit/propsschema and the property table below.