Project

General

Profile

Actions
Copy link

Bug #72957

closed

Unclear position for the new page using the wizard

Added by Christian Toffolo about 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Michael Oehlhof
Category:
Backend User Interface
Target version:
Candidate for patchlevel
Start date:
2016-01-27
Due date:
% Done:

100%

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

Description

TYPO3 7.6
Editor uses the new page wizard and get confused on where the new page will be created looking at the tree of "Select a position for the new page:"
The indentation of the pages in the tree is wrong showing pages and sub-pages on the same level.

TYPO3 6.2 was simply right and clear.

Files

Download all files
Capture.PNG (38.9 KB) Capture.PNG Christian Toffolo, 2016-01-27 10:49
Capture2.PNG (29.7 KB) Capture2.PNG Christian Toffolo, 2016-01-27 10:52
cattura.png (15.7 KB) cattura.png Riccardo De Contardi, 2016-01-28 10:23
cattura1.png (16.1 KB) cattura1.png Riccardo De Contardi, 2016-06-13 14:33
cattura2.png (48.4 KB) cattura2.png Riccardo De Contardi, 2016-06-13 14:33
newpagewizard.png (57.1 KB) newpagewizard.png Christian Toffolo, 2016-06-13 14:35
Schermata 2016-10-23 alle 10.15.01.png (56.2 KB) Schermata 2016-10-23 alle 10.15.01.png Riccardo De Contardi, 2016-10-23 10:22

Related issues 1 (0 open — 1 closed)

Related to TYPO3 Core - Bug #76638: Insertion point not correctly displayed when creating a sub page via context menuClosed2016-06-15

Actions
Actions
Copy link
 #1

Updated by Christian Toffolo about 8 years ago

This is how the wizard looks in 4.7
I don't have a 6.2 actually but if I remember well was pretty the same.

Actions
Copy link
 #2

Updated by Riccardo De Contardi about 8 years ago

  • Category set to Backend User Interface
Actions
Copy link
 #3

Updated by Riccardo De Contardi about 8 years ago

I add a further weird behavior (I tested witht the latest master 8.0-dev):
As you can see from my screenshot Cattura.png, there is an arrow inside the page with id=339 ("Prova 32") that seems to create the page inside that page, but... it actually creates it below the page, at the same level

Actions
Copy link
 #4

Updated by Riccardo De Contardi about 8 years ago

  • Target version changed from 7.6.3 to Candidate for patchlevel
Actions
Copy link
 #5

Updated by Frank Nägler almost 8 years ago

  • Status changed from New to Needs Feedback

can't reproduce the bug with latest 7.6 and master branch.
Is it still an issue?

Actions
Copy link
 #6

Updated by Christian Toffolo almost 8 years ago

I just checked with latest 7.6 and bug is still present.

Actions
Copy link Download all files
 #7

Updated by Riccardo De Contardi almost 8 years ago

In fact, IMO it is reproducible on 8.2-dev (I have not tested with 7.6, but I think that's the same)

steps to reproduce:
a) create a page named "72957", and create some subpages ("Alpha", "Beta", "Gamma"), so you have the situation illustrated on cattura1.png
b) select the page 72957"; list view; create new record; Create a new page (select position)
c) the result is shown in cattura2.png:

1) it is not clear from the position of the pages and the arrows, that the pages Alpha, Beta, Gamma are in fact inside 72957
2) looking at the last arrow, it seems that it creates a page inside Gamma, but it actually creates it after Gamma

Actions
Copy link
 #8

Updated by Christian Toffolo almost 8 years ago

This is the actual screenshot of the latest commit of 7.6 from git with some note added to help to understand the bug.

Actions
Copy link
 #9

Updated by Michiel Roos over 7 years ago

This was broken in commit https://github.com/TYPO3/TYPO3.CMS/commit/2affe49148755ec49731dc24de47b452e8919437

Due to changes in the method: \TYPO3\CMS\Backend\Tree\View\AbstractTreeView::getTree()

Actions
Copy link
 #10

Updated by Tymoteusz Motylewski over 7 years ago

  • Status changed from Needs Feedback to New

confirmed in v8.3

Actions
Copy link
 #11

Updated by Michael Oehlhof over 7 years ago

  • Assignee set to Michael Oehlhof
Actions
Copy link
 #12

Updated by Gerrit Code Review over 7 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/50322

Actions
Copy link
 #13

Updated by Gerrit Code Review over 7 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/50322

Actions
Copy link
 #14

Updated by Christian Toffolo over 7 years ago

Patch works for me in TYPO3 7.6.11
Thank you Michael Oehlhof!

Actions
Copy link
 #15

Updated by Riccardo De Contardi over 7 years ago

Still not totally clear: if you are in the situation described by the attached screenshot Schermata 2016-10-23 alle 10.15.01.png and click on the arrow circled, the created page will not be a subpage of the page, but a sibling.

Actions
Copy link
 #16

Updated by Gerrit Code Review over 7 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/50322

Actions
Copy link
 #17

Updated by Gerrit Code Review over 7 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/50322

Actions
Copy link
 #18

Updated by Gerrit Code Review over 7 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/50322

Actions
Copy link
 #19

Updated by Gerrit Code Review over 7 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50396

Actions
Copy link
 #20

Updated by Michael Oehlhof over 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #21

Updated by Christian Toffolo over 7 years ago

I tested https://review.typo3.org/#/c/50396/ and this bug doesn't seem to be resolved.
As I commented here https://review.typo3.org/#/c/50322/:
"If you try to create a page from a page that has no children, the arrow that signal the position for the sub-page of this page is not indented."

Actions
Copy link
 #22

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF