Bug #80352

RecordList: Delete record: referenced records don't have a fancy delete fade out animation

Added by Josef Glatz over 2 years ago. Updated 2 months ago.

Status:
Accepted
Priority:
Should have
Assignee:
Category:
Backend User Interface
Start date:
2017-03-20
Due date:
% Done:

0%

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

Description

Problem/Description

Scenario: I delete a record in recordlist, where also child elements are deleted. After agreeing the modal box, only the deleted record is removed (animation) from the table. Referenced records (of other tables) (in my example on the same pid) are correctly removed, but an editor have to reload the frame/module. Something can lead to confusion for an TYPO3 backend editor.

You can test this with EXT:powermail > v2: Create a form with pages and fields. Than delete the form in recordlist.

Suggestion

As I actually don't know how complex the deletion via ajax is: what about forcing a reload, when there is a case like this (referenced items or child items?!)

ToDo(s)

  • ?

Acceptance Criteria

  • ?

History

#1 Updated by Josef Glatz over 2 years ago

  • Subject changed from RecordList: Delete record: referenced records aren't deleted to RecordList: Delete record: referenced records don't have a fancy delete fade out animation

#2 Updated by Benni Mack about 2 years ago

  • Target version changed from 8 LTS to next-patchlevel

#3 Updated by Oliver Hader about 2 years ago

  • Status changed from New to Accepted

The drawback is, that the AJAX invocation results in an implicit DOM update for the clicked element. Reloading the whole document would mitigate the desired impact of using an AJAX call at all.

Two possibilities:
  • model based client approach (MVVM style with data-binding)
  • modify AJAX response to return imperatives for a set of CSS selectors (e.g. "remove": [".t3js-entity[...]", ".t3js-entity[...]"])

#4 Updated by Oliver Hader about 2 years ago

  • Assignee set to Oliver Hader
  • Tags set to JavaScript

#5 Updated by Benni Mack 2 months ago

  • Target version changed from next-patchlevel to Candidate for patchlevel

Also available in: Atom PDF