Bug #25157

Page module + backend layout - title of unassigned columns

Added by Tomas Mrozek over 10 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2011-02-24
Due date:
% Done:

100%

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

Description

If a column has no "colPos" set, then it's rendered in the Page module in a way that new content cannot be added to this column and it's title is "Not assigned". It's obviously correct that the column is inaccessible, however as long as the column has a title it shall still be displayed (instead of "Not assigned"). Only columns having no "colPos" and no title shall have "Not assigned" title instead.

Even columns which cannot be assigned might still have some reason and a BE user shall be given a clue which column it is that cannot be used.

This way the developer has more control over what is shown to the BE users in the Columns view of Page module.

(issue imported from #M17741)


Files

17741.patch (797 Bytes) 17741.patch Administrator Admin, 2011-02-24 15:31

Related issues

Related to TYPO3 Core - Bug #45550: Having colPos 0 and columns not assigned multiplies elements in page module viewClosedJo Hasenau2013-02-17

Actions
#1

Updated by Tomas Mrozek over 10 years ago

I consider this a bug because if a column title is set, it shall be displayed.

#2

Updated by Tomas Mrozek over 10 years ago

Patch suggestion uploaded.

#3

Updated by Jo Hasenau over 10 years ago

I had some time in the train on my way back home today so I was able to fix this - will come up with a patch until tomorrow since I have to separate it from the other grid related patches first.

#4

Updated by Jo Hasenau over 10 years ago

Just compared your patch with mine and it's pretty much the same thing, so I will make an RFC of it now.

#5

Updated by Jo Hasenau over 10 years ago

Pending in core list

#6

Updated by Ernesto Baschny almost 8 years ago

  • Target version deleted (0)
  • Is Regression set to No

Joey, do you still have that patch? Is this still relevant? Maybe rework it as a review request?

#7

Updated by Jo Hasenau almost 8 years ago

Will take care of that.

Maybe we can do it next weekend during the Gridelements workshop.

#8

Updated by Philipp Gampe almost 8 years ago

  • Status changed from New to Closed

Duplicate of #45550.

#9

Updated by Philipp Gampe almost 8 years ago

  • Status changed from Closed to Accepted

My fault, this is a valid concern.

#10

Updated by Gerrit Code Review almost 8 years ago

  • Status changed from Accepted to Under Review

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18349

#11

Updated by Gerrit Code Review almost 8 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18349

#12

Updated by Gerrit Code Review almost 8 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18349

#13

Updated by Gerrit Code Review over 7 years ago

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

#14

Updated by Gerrit Code Review over 7 years ago

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

#15

Updated by Gerrit Code Review over 7 years ago

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

#16

Updated by Gerrit Code Review over 7 years ago

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

#17

Updated by Georg Ringer over 7 years ago

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

Updated by Gerrit Code Review over 7 years ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch TYPO3_4-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25388

#19

Updated by Gerrit Code Review over 7 years ago

Patch set 2 for branch TYPO3_4-5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25389

#20

Updated by Philipp Gampe over 7 years ago

  • Status changed from Under Review to Resolved
#21

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF