Replies: 4 comments
-
|
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hey 👋
While refactoring this library to introduce component prefix and better linting I wen't through every single file... and since I was updating components as well as docs I have few thoughts about implementation, styling, usage of props, errors and the other more or less opinionated things...
Here is the "raw" list of issues I found during the above process... some would require separate tickets/discussions... some just decision which way to go with them.
When responding please use "Issue number" and if we'll create dedicated tickets... I'll edit this message to link them properly.
heading should be H element<router-link>
if propto
is setflowbite-svelte
/* eslint-disable
avatar-stack-counter
shouldn't be a linkavatar-stack-counter
total
doesn't workbreadcrumb-item
should be used internally andbreadcrumbs
component should accept arraybreadcrumb-item
should be<router-link>
if propto
is set<Accordion :open-first-item="false">
should be just<Accordion first-item-closed>
slide
prop is not workinginterval
prop should not be slide dependentright
andleft
are working on second click (bad position on first one)fwb-list-group-item
hover only makes sense if it's clickable (should accepthref
orto
)pagination
usebutton-group
internally?disabled pagination button should not have hover effect ("Previous" / "Next")FIX: refactor: Update disabledClasses in FwbPagination.vue #319besideText
slot in rating component should have default stylingw-${size} h-${size}
directive
should be boolean and named likedirective-show
(easier DX)click:pane
should pass through event datatype="empty"
shouldn't be needed (without-icon
would make more sense and allow for more flexibility)flowbite-themable
componentfile-input
should haveaccept
prop to allow for file extensions filteringfile-input
withdeopzone="true"
doesn't accept multiple filesselect
have conflicting stylesselect
dropdown is not always styled properly ("placeholder sometimes disappears")textarea
comment example<fwb-button color="">
or<fwb-button color>
looks like a hack and needs to be fixedselect-none
so it's not selectable on multiple clicksrange
component label'sfor
attribute can't be hard-codedfwb-button
instead of HTML onesize
with different option.. and sometimes we have dedicated prop likelarge
(pagination)Failed to resolve component: router-link
in Sidebar component (we need to add vue router dependency for that to work)file upload
component button is too dark in light themeBeta Was this translation helpful? Give feedback.
All reactions