Feat/color combobox #267
Feat/color combobox #267
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
jurgenbelien
force-pushed
the
feat/color-combobox
branch
from
ba9633d to
d12b527
Compare
jurgenbelien
force-pushed
the
feat/color-combobox
branch
from
d12b527 to
ec11ebb
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #267 +/- ##
===========================================
- Coverage 100.00% 94.09% -5.91%
===========================================
Files 12 45 +33
Lines 694 1254 +560
Branches 100 264 +164
===========================================
+ Hits 694 1180 +486
- Misses 0 61 +61
- Partials 0 13 +13 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
jurgenbelien
force-pushed
the
feat/color-combobox
branch
from
ec11ebb to
7d3bfe0
Compare
… stylesheet is not loaded, query listbox before intersectionObserver
jurgenbelien
force-pushed
the
feat/color-combobox
branch
from
7d3bfe0 to
98fd609
Compare
jurgenbelien
force-pushed
the
feat/color-combobox
branch
from
f3eca99 to
2620c10
Compare
|
| // decorators: [StoryRootDecorator], | ||
| initialGlobals: { | ||
| storyRootClassname: 'voorbeeld-theme', | ||
| storyRootClassname: 'ma-theme', |
There was a problem hiding this comment.
werkt dit? ik dacht dat storyRootClassname niet bestond
There was a problem hiding this comment.
Ik heb t niet opgezocht tbh, die storyRootClassname stond er, en ik zie nu n ma-theme hier in de body van de preview iframe
|
|
||
| async loadLocalizations(lang: string) { | ||
| const SEPARATOR = '-'; | ||
| const subtags = lang.split(SEPARATOR); |
There was a problem hiding this comment.
Ik weet niet 100% zeker of ik de context goed heb, maar je kunt ook Intl.Locale gebruiken om de language en region te pakken:
// lang = 'en-US';
const locale = new Intl.Locale(lang)
const region = locale.region; // 'US'
const language = locale.language; // 'en'There was a problem hiding this comment.
M'n gedachte was "stel nu er is een woord (of kleurnaam) in een taal hetzelfde, maar een locale of schrift heeft daar een uitzondering op", dan wil je naast je <language>.ts file ook een <language>-<region>.ts of een <language>-<region>-<script>.ts hebben die je dan inlaadt. Voor specifiek die cascade zit die functie erin. Ik kan m expliciet maken met het eerst via de Intl.locale doen of doen totdat de streepjes op zijn. Dat eerste is misschien wel duidelijker ja 😅
| }, | ||
| "scripts": { | ||
| "dev": "pnpm run storybook", | ||
| "dev": "pnpm run storybook --no-open", |
There was a problem hiding this comment.
Fair warning: losse PR gemaakt want ik trok het niet meer #287
No description provided.