Project

General

Profile

Actions

Bug #66541

closed

Content droppable in dropzones outside of the viewport

Added by Jo Hasenau over 9 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2015-04-22
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When using drag & drop the drop zones before or after elements are activated even when the dropzone itself is outside of the view port.
This leads to unwanted move actions, when the user stops dragging the element around without noticing the dropzone that is currently active.

IMHO dropzones should not be activated when hovering a content element but when hovering the dropzone itself only.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #66542: Drag & Drop can not be stopped without really dropping an element into a dropzoneClosedJo Hasenau2015-04-22

Actions
Related to TYPO3 Core - Bug #65311: Moving records in page module "hangs"ClosedAndreas Kienast2015-02-25

Actions
Actions #1

Updated by Jo Hasenau over 9 years ago

Same issue here as with #66542: The change from draggable to sortable introduced the current behaviour. Instead of using the drop zones as trigger, now hovering the elements themselves triggers the dropzones. So a larger element with a lot of preview content inside can trigger a dropzone which is not visible in the viewport.

Actions #2

Updated by Gerrit Code Review over 9 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38861

Actions #3

Updated by Gerrit Code Review over 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38861

Actions #4

Updated by Gerrit Code Review over 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38861

Actions #5

Updated by Gerrit Code Review over 9 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38861

Actions #6

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #7

Updated by Gerrit Code Review over 9 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #8

Updated by Gerrit Code Review over 9 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #9

Updated by Gerrit Code Review over 9 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #10

Updated by Gerrit Code Review over 9 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #11

Updated by Gerrit Code Review over 9 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #12

Updated by Gerrit Code Review over 9 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #13

Updated by Gerrit Code Review over 9 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/38865

Actions #14

Updated by Jo Hasenau over 9 years ago

Can be fixed by switching back to draggable instead of sortable.

Sortable should be used for the list module, while the page module should be switched back to draggable.

Actions #15

Updated by Benni Mack over 9 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
Actions #16

Updated by Daniel Goerz over 9 years ago

  • Status changed from Under Review to Needs Feedback

Since the patch was abandoned, can this issue be closed or does it need a new patch?

Actions #17

Updated by Jo Hasenau over 9 years ago

Well - there is #67134 which provides a solution for this problem, but maybe we should cut that part off that issue and provide a separate patch for it here.

The actual problem has been the switch from droppable to sortable, which changes some behaviours to the worse, while some other problems have been fixed by it.
I discussed that already with Andreas Fernandez and we came to the conclusion, that sortable should be used for list like structures only, while the page module should be switched back to droppable.

To finally have structure containers in the core, we will have to switch to droppable anyway, since this would be a real PITA for editors with sortable.
So maybe it would be a good idea to change D&D now, while containers will have to wait for upcoming versions.

Actions #18

Updated by Susanne Moog over 9 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #19

Updated by Benni Mack about 9 years ago

  • Target version deleted (7.5)
Actions #20

Updated by Alexander Opitz almost 9 years ago

  • Status changed from Needs Feedback to New
  • TYPO3 Version changed from 7 to 8
Actions #21

Updated by Riccardo De Contardi over 7 years ago

  • Status changed from New to Closed
  • Assignee deleted (Jo Hasenau)

The core is back on Droppable with CMS 8: the dropzones become green and are activated when an element is placed upon them
so I think that this could be safely closed.

If you experience the issue again then please reopen it or open a new issue with a reference to this one. Thank you

Actions #22

Updated by Coders.Care Extension Team almost 7 years ago

  • Related to deleted (Task #67134: The core must provide structured content elements which are configurable similar to backend layouts out of the box)
Actions

Also available in: Atom PDF