collapse icon indicating copy to clipboard operation
collapse copied to clipboard

Add an unset option to the collapse active key

Open Ahmed-Sarkawt opened this issue 4 years ago • 4 comments

Having an unset option to provide easier customization to the collapse

Ahmed-Sarkawt avatar Nov 30 '20 07:11 Ahmed-Sarkawt

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/react-component/collapse/6qe4in9iw
✅ Preview: https://collapse-git-patch-1.react-component.vercel.app

vercel[bot] avatar Nov 30 '20 07:11 vercel[bot]

Codecov Report

Merging #180 (94b6195) into master (ea1ab91) will decrease coverage by 0.91%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
- Coverage   99.04%   98.13%   -0.92%     
==========================================
  Files           4        4              
  Lines         105      107       +2     
  Branches       33       34       +1     
==========================================
+ Hits          104      105       +1     
- Misses          1        2       +1     
Impacted Files Coverage Δ
src/Collapse.tsx 98.43% <66.66%> (-1.57%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ea1ab91...94b6195. Read the comment docs.

codecov[bot] avatar Nov 30 '20 07:11 codecov[bot]

Having an unset option to the activeKey prop in order to make the component easier to handle

Ahmed-Sarkawt avatar Nov 30 '20 08:11 Ahmed-Sarkawt

This pull request introduces 12 alerts when merging 06c92a32115457cfdcec6fb05c357f782f6939b7 into ea1ab9196e8721e90cd40af8a82a950d4b7dab06 - view on LGTM.com

new alerts:

  • 12 for Syntax error

lgtm-com[bot] avatar Nov 30 '20 08:11 lgtm-com[bot]