Bug #35223

Whitespace is trimmed from pageTitleSeparator

Added by Chris topher about 1 year ago. Updated 2 months ago.

Status:Resolved Start date:2012-03-26
Priority:Must have Due date:
Assignee:- % Done:

100%
Category:TypoScript
Target version:-
TYPO3 Version:4.7 Complexity:
PHP Version:
Votes: 4 (View)

Description

#17030 introduced a way to use a custom separator in the title tag of the generated website. This should allow two use cases:
  • A way to create a title tag like "website - page title", e.g. with config.pageTitleSeparator = (whitespace)-(whitespace).
  • A way to create "website*page title".

However, neither of this is currently working.

As far as I see there is no way to preserve whitespaces at the start of the separator. The TS parser will trim leading whitespaces, which are provided in the TS configuration. So all you get is "website- page title". This makes the first use case impossible.
A whitespace at the end of the separator is always added; this is hardcoded in the PHP code. This makes the second use case impossible.

Coding the whitespaces with nbsp tags does not help either, because the title tag later is htmlspecialchar'ed (so the tags would be displayed as "nbsp" in the title). stdWrap and/or optionSplit are not available for config.pageTitleSeparator.

Long story short:
A way to preserve or remove leading and trailing whitespaces in the separator is missing.

Related issues

related to TSref - Feature #46390: Add documentation of stdWrap to config.pageTitleSeparator Closed 2013-03-17
follows TSref - Task #32320: Add documentation of config.pageTitleSeparator Closed 2011-12-06
follows Core - Feature #17030: Integrate config.pageTitleSeparator Resolved 2007-02-23

Associated revisions

Revision 3bd2b3c4
Added by Georg Ringer 2 months ago

[FEATURE] Add stdWrap to config.pageTitleSeperator

Add stdWrap to config.pageTitleSeperator

Change-Id: I88c2141167414b184f158d09f40df3aa36390b4a
Resolves: #35223
Releases: 6.1
Reviewed-on: https://review.typo3.org/18123
Reviewed-by: Alexander Opitz
Tested-by: Alexander Opitz
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack

History

#1
Updated by Markus Klein about 1 year ago

Hi!

Do you think it would suffice to add stdWrap to config.pageTitleSeparator?

Example:

config.pageTitleSeparator = -
config.pageTitleSeparator.noTrimWrap = | | |

#2
Updated by Chris topher about 1 year ago

Hi Markus,

yes, that would solve the problem. (And the hardcoded space at the end of the separator has to be removed.)

#3
Updated by Andreas Wolf about 1 year ago

  • Subject changed from #17030 introduced a bug to Whitespace is trimmed from pageTitleSeparator
  • Status changed from New to Accepted

#4
Updated by Steffen Ritter about 1 year ago

  • Target version deleted (4.7.0)

#5
Updated by Stephan Großberndt 8 months ago

Without the stdWrap this long awaited feature is quite useless. Would it be possible to include it in the next update 4.7.5?

#6
Updated by Gerrit Code Review 3 months ago

  • Status changed from Accepted to Under Review

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

#7
Updated by Gerrit Code Review 3 months ago

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

#8
Updated by Georg Ringer 2 months ago

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

Also available in: Atom PDF