-
Couldn't load subscription status.
- Fork 7
fix: add routable main collections home page #10
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
Conversation
|
| <SidebarHeader className="border-border m-0 size-auto! border-b pl-6 h-auto! w-full!"> | ||
| <div className="flex h-[68px]! items-center gap-x-4"> |
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.
recheck ให้หน่อย ทำไมต้อง important
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.
เพราะว่า จากการเรียกใช้ Component SidebarHeader
<SidebarHeader className="border-border m-0 border-b pl-6 h-auto w-full!">...</SidebarHeader>ภายใน Component มีการเรียกใช้
const header = tv({
base: 'mb-2 flex flex-col **:data-[slot=sidebar-label-mask]:hidden',
variants: {
collapsed: {
false: '',
true: 'mt-2 group-data-[sidebar-intent=float]/sidebar-container:mt-2 md:mx-auto md:size-9 md:items-center md:justify-center md:p-0 md:hover:bg-(--sidebar-accent)',
},
},
})- ที่มาจากตัว Library ตรง
md:size-9ทำให้การกำหนดh-autoธรรมดาไม่สามารถทำได้ ถูกsize-9override ไป - ที่มาจากตัว Library ตรง
md:size-9ทำให้การกำหนดw-fullธรรมดาไม่สามารถทำได้ ถูกsize-9override ไป เช่นกัน - จาก
h-[68px]!สามารถเปลี่ยนเป็นh-[68px]ได้ครับ ✅
จริง ๆ สามารถเอา md:size-9 ออกได้จาก SidebarHeader ของ IntentUI แต่คิดว่าไม่ควรไปหยุมหยัมกับสิ่งที่ Library ให้มาเท่าไหร่ จะให้เอาออกไหมครับ
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.
ถ้าเป็น h-auto md:h-auto w-full md:w-full จะ work มั้ยนะ
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.
ส่วนอันข้างล่าง h-[68px] เฉย ๆ น่าจะ work นะ?
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.
Other than that, LGTM
I'd like to suggest creating a
layoutsdirectory underviews/collectionsand move layout falling under the collections toviews/collections/layoutsfolder