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: Dragging a card to another column then back doesn't work #773

Closed
chenilim opened this issue Jul 23, 2021 · 9 comments
Closed

Bug: Dragging a card to another column then back doesn't work #773

chenilim opened this issue Jul 23, 2021 · 9 comments
Assignees
Labels
Bug Something isn't working Regression This is a regression from previously working functionality in a released version

Comments

@chenilim
Copy link
Contributor

Summary:
Steps to reproduce the behavior:

  1. Create a new board, e.g. using the Roadmap template
  2. Drag a card from column A to column B
  3. Drag the card back from column B to column A
  4. Nothing happens

Expected behavior:
The card returns to column A

Screenshots:
https://user-images.githubusercontent.com/46905241/126744015-70847796-e499-4f78-8e2a-af271a3e713e.mov

Platform:

  • Focalboard 0.8.1 on Mattermost v5.37 (community server)
  • Chroms 91 on macOS

Additional context:
If you first drag another card to column A, then drag the original card to column A, that works.

@chenilim chenilim added Bug Something isn't working Regression This is a regression from previously working functionality in a released version labels Jul 23, 2021
@chenilim
Copy link
Contributor Author

@harshilsharma63, could you help investigate this one too? Not sure if this is already fixed in main. If so, please also fix on the release-0.8.0 branch.

I have noticed some general instability with sequential drag operations like this one.

@chenilim
Copy link
Contributor Author

Is this a dup of #708? Please close if so, and cherry pick / verify that fix on release-0.8.0. Thanks.

@harshilsharma63
Copy link
Member

@chenilim yes it's a duplicate of #708 and is fixed on main. Can you give it another try?

@chenilim
Copy link
Contributor Author

Thanks Harshil. If it's fixed in main, please cherry-pick or re-fix on release-0.8.0. Thanks.

@harshilsharma63
Copy link
Member

@chenilim done. Cherrypicked into release-0.8.0.

@hahmadia
Copy link
Contributor

Since #708 is close should we close this too?

@harshilsharma63
Copy link
Member

@hahmadia yes. @chenilim can you just try it out on the latest release-0.8.0 branch? I just want to make sure I got it.

@chenilim
Copy link
Contributor Author

Thanks @harshilsharma63! I'm out until next week. @ogi-m, can you help Harshil test this? Thanks.

@harshilsharma63 harshilsharma63 added the 2: QA Review Requires review by a QA tester label Jul 28, 2021
@ogi-m
Copy link
Contributor

ogi-m commented Jul 28, 2021

✔️ Confirmed this is now fixed - no issues with when moving cards between columns

@ogi-m ogi-m closed this as completed Jul 28, 2021
@ogi-m ogi-m removed the 2: QA Review Requires review by a QA tester label Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Regression This is a regression from previously working functionality in a released version
Projects
None yet
Development

No branches or pull requests

4 participants