Project

General

Profile

Actions

Task #89220

closed

Revert value of input if confirmation modal is not accepted

Added by Harald Atteneder over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2019-09-21
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

When changing the doktype a confirmation modal appears. After closing the modal without clicking OK the new value is still selected. To improve UX it would be better to revert to the previous value.


Related issues 3 (2 open1 closed)

Related to TYPO3 Core - Feature #34624: Changing the type field of a new element should not result in saving Accepted2012-03-07

Actions
Related to TYPO3 Core - Feature #89361: Changing 'onChange' => 'reload' behaviorNew2019-10-07

Actions
Related to TYPO3 Core - Task #89362: Revert changed backend behavior for onChange/typeClosed2019-10-07

Actions
Actions #1

Updated by Riccardo De Contardi over 4 years ago

This seems related #34624

Actions #2

Updated by Gerrit Code Review over 4 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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/61771

Actions #3

Updated by Gerrit Code Review over 4 years ago

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

Actions #4

Updated by Gerrit Code Review over 4 years ago

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

Actions #5

Updated by Gerrit Code Review over 4 years ago

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

Actions #6

Updated by Harald Atteneder over 4 years ago

  • Related to Feature #34624: Changing the type field of a new element should not result in saving added
Actions #7

Updated by Harald Atteneder over 4 years ago

  • % Done changed from 0 to 100
Actions #8

Updated by Gerrit Code Review over 4 years ago

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

Actions #9

Updated by Gerrit Code Review over 4 years ago

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

Actions #10

Updated by Gerrit Code Review over 4 years ago

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

Actions #11

Updated by Gerrit Code Review over 4 years ago

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

Actions #12

Updated by Gerrit Code Review over 4 years ago

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

Actions #13

Updated by Gerrit Code Review over 4 years ago

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

Actions #14

Updated by Gerrit Code Review over 4 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61771

Actions #15

Updated by Harald Atteneder over 4 years ago

  • Status changed from Under Review to Resolved
Actions #16

Updated by Gerrit Code Review over 4 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61809

Actions #17

Updated by Gerrit Code Review over 4 years ago

Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61809

Actions #18

Updated by Harald Atteneder over 4 years ago

  • Status changed from Under Review to Resolved
Actions #19

Updated by Tobi Kretschmann over 4 years ago

  • Status changed from Resolved to Accepted
Actions #20

Updated by Tobi Kretschmann over 4 years ago

  • Related to Feature #89361: Changing 'onChange' => 'reload' behavior added
Actions #21

Updated by Tobi Kretschmann over 4 years ago

Reverted this patch because we need a more general approach to fix this issue.
See: https://forge.typo3.org/issues/89361

Actions #22

Updated by Oliver Hader over 4 years ago

  • Related to Task #89362: Revert changed backend behavior for onChange/type added
Actions #23

Updated by Susanne Moog about 4 years ago

  • Status changed from Accepted to Closed

Is now a duplicate of #89361 - see previous comments. Let's continue there.

Actions

Also available in: Atom PDF