New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Add IconV2 with new svg settings and revert old Icons changed by PR #384 #387
base: main
Are you sure you want to change the base?
Conversation
…at least one keyboard listener
7492b0f
to
05d8e60
Compare
Quality Gate passedIssues Measures |
✅ Knip Scan Result for 9d1aaa8 Unused files (9)
Unused dependencies (2)
Unused devDependencies (5)
Unused exported types (2)
Unused exported types in namespaces (1)
Configuration issues (1)
|
🤖 Lighthouse Scan Result for 9d1aaa8 /rooms
/rooms/abc
/login
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
幫我把 storybook 白底白字的部分調整好就可以了~
其他看起來 ok
|
||
return ( | ||
<> | ||
<p className="absolute top-[-1.25rem] right-0 text-white/90"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
這裡的 className 幫我調整成 absolute top-0 right-4 text-zinc-950
讓 storybook 的白畫面可以看的到字
label="搜尋" | ||
value={value} | ||
onChange={setValue} | ||
labelClassName="mr-3 leading-normal" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
labelClassName 也幫我改成 mr-3 leading-normal text-zinc-950
讓字顯示出來
onClick={handleClick(iconName)} | ||
> | ||
<IconV2 name={iconName} className="w-full h-full" /> | ||
<p className="text-center text-white/90 whitespace-nowrap"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
這裡的 className 幫我改成 text-center text-zinc-950 whitespace-nowrap
讓字顯示出來
Why need this change? / Root cause:
Changes made:
Test Scope / Change impact:
Issue