Skip to content
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

Migrate to react dnd #27

Open
wants to merge 3 commits into
base: clerk-authentication
Choose a base branch
from

Conversation

eyob-muktar
Copy link

resolves #21

Copy link

vercel bot commented Dec 22, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @vishwajeetraj11 on Vercel.

@vishwajeetraj11 first needs to authorize it.

@eyob-muktar
Copy link
Author

I only updated the ReorderEducation file. If the problem gets fixed I'll update the rest.

@vishwajeetraj11
Copy link
Owner

vishwajeetraj11 commented Dec 26, 2023

Hey @eyob-muktar, the library is accounting for 60px of extra space because of this code (EducationCard.js line no. 23).

style={{ maxHeight: `${educationActive[id] ? '60px' : '0px'}`, transition: 'all 0.5s', overflow: 'hidden' }}

See if there is a work around.

@eyob-muktar
Copy link
Author

I'm busy at the moment I'll check when I get free time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use react-dnd kit instead of react-beautiful-dnd
2 participants