Project

General

Profile

Actions

Bug #90777

closed

Trying to edit an existing file mount in BE group form results in error

Added by Sybille Peters over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2020-03-18
Due date:
% Done:

100%

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

Description

The question is: How to edit (existing) file mount in be groups form in BE?
I don't know how to do it. If there is a way to do it, perhaps there is a way to make this more intuitive.

Reproduce

  1. Open form for editing a group
  2. There are some existing file mounts in the list of "Available items"
  3. There is (AFAIK) no way to select one without moving it to the list "Selected items", but selecting it and then removing it again supposedly make it selected in the list "Available Items"
  4. click the pencil "Edit filemount"

Actual outcome

Even if an item is selected, clicking on the edit pencil yields:

"Please select one or more items in the list before you can edit."

Expected outcome

The edit pencil "Edit filemount" should edit the selected item


Files

filemount.png (23.9 KB) filemount.png Sybille Peters, 2020-03-18 12:40
step1.png (85.8 KB) step1.png Susanne Moog, 2020-03-19 11:23
Actions #1

Updated by Susanne Moog over 4 years ago

  • Status changed from New to Accepted

I think the expected behaviour should be different, however I agree that we have a UI problem here:

It's totally fine IMHO that the edit pencil only allows editing of active items in this context, as it doesn't make much sense to go through the length of opening this group record to edit an unrelated group - and in fact is the source of the problem here, as the data is taken from the active records field with that value. --> Selecting it by first selecting than unselecting is also kind of awkward.

However the placement of the pencil on the right of the field does suggest the editing of all records and should probably be moved.

Actions #2

Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted 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/+/63792

Actions #3

Updated by Susanne Moog over 4 years ago

Actions #4

Updated by Sybille Peters over 4 years ago

Yes, I agree. Removing the pencil would be sufficient. There are other places to edit the filemounts (e.g. on page 0).

Actions #5

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/+/63792

Actions #6

Updated by Susanne Moog over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Benni Mack over 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF