Skip to content

Conversation

@MateoWartelle
Copy link
Contributor

@MateoWartelle MateoWartelle commented Sep 17, 2023

Summarize the changes made and the motivation behind them.

Kbd component to match existing flowbite library

Reference related issues using # followed by the issue number.

If there are breaking API changes - like adding or removing props, or changing the structure of the theme - describe them, and provide steps to update existing code.

@vercel
Copy link

vercel bot commented Sep 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
flowbite-react ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 21, 2023 3:49pm
@codecov
Copy link

codecov bot commented Sep 17, 2023

Codecov Report

Patch coverage: 71.68% and project coverage change: -6.03% ⚠️

Comparison is base (7461173) 99.54% compared to head (b8ce99c) 93.52%.
Report is 78 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@ ## main #969 +/- ## ========================================== - Coverage 99.54% 93.52% -6.03%  ========================================== Files 163 182 +19 Lines 6621 7892 +1271 Branches 401 439 +38 ========================================== + Hits 6591 7381 +790  - Misses 30 511 +481 
Files Changed Coverage Δ
src/components/Pagination/helpers.ts 100.00% <ø> (ø)
src/components/Toast/theme.ts 100.00% <ø> (ø)
src/components/Datepicker/Datepicker.tsx 28.43% <28.43%> (ø)
src/components/Datepicker/Views/Days.tsx 36.00% <36.00%> (ø)
src/components/Datepicker/Views/Decades.tsx 37.70% <37.70%> (ø)
src/components/Datepicker/Views/Months.tsx 37.70% <37.70%> (ø)
src/components/Datepicker/Views/Years.tsx 37.70% <37.70%> (ø)
src/components/Banner/BannerCollapseButton.tsx 40.90% <40.90%> (ø)
src/components/Datepicker/helpers.ts 60.13% <60.13%> (ø)
src/components/Banner/Banner.tsx 64.70% <64.70%> (ø)
... and 45 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MateoWartelle MateoWartelle changed the title Feat/kbd component to match flowbite library feat : KBD component Sep 17, 2023
@MateoWartelle
Copy link
Contributor Author

I am going to write some tests and it should be ready for review after. I am open to change whatever is suggested.

thanks

@MateoWartelle MateoWartelle marked this pull request as ready for review September 17, 2023 20:45
@rluders
Copy link
Collaborator

rluders commented Sep 19, 2023

@MateoWartelle please, do not use merge, use rebase instead.

<CodePreview
importFlowbiteReact="Kbd"
importExternal="import { MdKeyboardArrowLeft, MdKeyboardArrowRight, MdKeyboardArrowDown, MdKeyboardArrowUp } from 'react-icons/md';"
title="Arrow Keys Kbd"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Arrow Keys KDB

@rluders
Copy link
Collaborator

rluders commented Sep 21, 2023

Looks nice, @MateoWartelle - just some small details that need to be fixed and we are good to get it merged.

@rluders rluders merged commit cd61cab into themesberg:main Sep 21, 2023
maman pushed a commit to transkripid/flowbite-react that referenced this pull request Oct 7, 2023
* feat: kBD (Keyboard) component * feat: add kbd story (default, withIcon) * feat: remove extra sentence + flex wrap letter keys * docs: fix docs * feat: kbd basic tests, storybook withIcon + onlyIcon * docs: add missing importExternal for arrow keys docs * docs: review changes for documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants