Bug #90777
closedTrying to edit an existing file mount in BE group form results in error
100%
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¶
- Open form for editing a group
- There are some existing file mounts in the list of "Available items"
- 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"
- 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
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.
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
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).
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
Updated by Susanne Moog over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 3503dfae0127347ca4219eb91b1ca06c1668c5ac.