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

Bug - Droppable - Empty Droppables do not receive Draggables #9749

Closed
bostrick opened this issue Oct 13, 2023 · 3 comments · Fixed by #10692
Closed

Bug - Droppable - Empty Droppables do not receive Draggables #9749

bostrick opened this issue Oct 13, 2023 · 3 comments · Fixed by #10692
Assignees
Labels
Milestone

Comments

@bostrick
Copy link

bostrick commented Oct 13, 2023

Describe the problem
When a "Droppable", as found in the "Multilist Example", is empty, it does not receive a Draggable.

https://www.patternfly.org/components/drag-and-drop/react/multiple-lists

How do you reproduce the problem?
Open the above link, move all items from right hand list to left hand list, and then attempt to move an item back.

Expected behavior
A empty destination Droppable, if given a min-height, should receive a Draggable, but doesn't.

Is this issue blocking you?
I've implemented an ugly workaround by adding a placeholder if a list is empty, and stripping it if it is not.

Screenshots
image

What is your environment?

  • OS: recent fedora (F38)
  • Browser: chrome
  • Version: google-chrome-stable-117.0.5938.149-1.x86_64

What is your product and what release date are you targeting?

Any other information?

@bostrick bostrick added the bug label Oct 13, 2023
@github-project-automation github-project-automation bot moved this to Needs triage in PatternFly Issues Oct 13, 2023
@bostrick bostrick changed the title Bug - [Component] - [short description] Bug - [Droppable] - [Empty Droppables do not receive Draggables] Oct 13, 2023
@bostrick bostrick changed the title Bug - [Droppable] - [Empty Droppables do not receive Draggables] Bug - Droppable - Empty Droppables do not receive Draggables Oct 13, 2023
@wise-king-sullyman wise-king-sullyman moved this from Needs triage to Backlog in PatternFly Issues Oct 17, 2023
@adamviktora adamviktora self-assigned this Nov 1, 2023
@adamviktora adamviktora moved this from Backlog to In Progress in PatternFly Issues Nov 1, 2023
@adamviktora adamviktora removed their assignment Nov 1, 2023
Copy link

stale bot commented Jan 9, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the wontfix label Jan 9, 2024
@stale stale bot closed this as completed Jan 26, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in PatternFly Issues Jan 26, 2024
@tlabaj tlabaj reopened this Jun 3, 2024
@github-project-automation github-project-automation bot moved this from Done to Needs triage in PatternFly Issues Jun 3, 2024
@tlabaj tlabaj added Pinned and removed wontfix labels Jun 3, 2024
@tlabaj tlabaj moved this from Needs triage to Backlog in PatternFly Issues Jun 3, 2024
@tlabaj tlabaj modified the milestones: Prioritized Backlog, 2024.Q2 Jun 4, 2024
@kmcfaul kmcfaul assigned kmcfaul and tlabaj and unassigned kmcfaul Jun 5, 2024
@kmcfaul kmcfaul moved this from Backlog to Not started in PatternFly Issues Jun 18, 2024
@nicolethoen nicolethoen moved this from Not started to In Progress in PatternFly Issues Jul 1, 2024
@tlabaj
Copy link
Contributor

tlabaj commented Jul 2, 2024

@bostrick we recommend that you use the flex layout instead of the split layout. I have pushed a PR to update our example code. you cna see that here #10692

@nicolethoen nicolethoen moved this from In Progress to PR Review in PatternFly Issues Jul 2, 2024
@github-project-automation github-project-automation bot moved this from PR Review to Done in PatternFly Issues Jul 16, 2024
@bostrick
Copy link
Author

Thanks all!

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

Successfully merging a pull request may close this issue.

5 participants