Project

General

Profile

Actions

Feature #21302

closed

click enlarge: popup window params are hardcoded

Added by Michael Sollmann over 14 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2009-10-19
Due date:
% Done:

100%

Estimated time:
PHP Version:
4.3
Tags:
Complexity:
Sprint Focus:

Description

the user cannot resize the popup or scroll the window e.g. to read long image captions. should be defined in typoscript.

(issue imported from #M12270)


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #16641: The "click-to-enlarge" function of the image content element does not provide scrollbars to the new window.Closed2006-10-12

Actions
Related to TYPO3 Core - Feature #31636: popupwindows have hardcoded window-name FEopenLinkClosed2011-11-07

Actions
Actions #1

Updated by Valery Romanchev over 14 years ago

related to #16641

Actions #2

Updated by Alexander Opitz almost 11 years ago

  • Category deleted (Communication)
  • Status changed from New to Needs Feedback
  • Target version deleted (0)

The issue is very old, does this issue exists in newer versions of TYPO3 CMS (4.5 or 6.1)?

Actions #3

Updated by Tobias Schmidt almost 11 years ago

This issue still exists in TYPO3 4.5. In sysext\cms\tslib\class.tslib_content.php, lines 1498 - 1502, it is not possible to insert something like "scrollbars=yes" via TypoScript. The parameters are hardcoded.

Actions #4

Updated by Alexander Opitz almost 11 years ago

  • Status changed from Needs Feedback to New
Actions #5

Updated by Mathias Schreiber about 9 years ago

  • Target version set to 7.2 (Frontend)
Actions #6

Updated by Benni Mack almost 9 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
Actions #7

Updated by Susanne Moog over 8 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #8

Updated by Benni Mack over 8 years ago

  • Target version deleted (7.5)
Actions #9

Updated by Riccardo De Contardi over 7 years ago

  • Category set to Frontend
Actions #10

Updated by Riccardo De Contardi over 5 years ago

Actually, you can add parameters to the popup using typoscript, with JSwindow_params

See documentation here:

https://docs.typo3.org/typo3cms/TyposcriptReference/Functions/Typolink/Index.html#jswindow-params

example:

page=PAGE
page.60=TEXT
page.60.value=hello world
page.60.typolink.parameter = 507 400x300
page.60.typolink.JSwindow_params=scrollbars=1,resizable=0,toolbar=0,status=0
page.60.wrap=<p>|</p>

The only "hardcoded" parameter seems to be the name "FEopenLink" - see #31636.

By the way

Modern browsers like Chrome or Firefox ignore the parameter resizable=0 ;)

Actions #11

Updated by Tobias Schmidt over 5 years ago

I'm not 100% sure but I think the TypoScript settings for JSwindow_params don't apply to the click enlarge context. In TYPO3 v8.7 the source code in method imageLinkWrap of class \TYPO3\CMS\Frontend\ContentObject\ContentObjectRenderer (lines 1468 - 1478) is basically the same as it was in TYPO3 v4.5:

$onClick = 'openPic('
    . GeneralUtility::quoteJSvalue($this->getTypoScriptFrontendController()->baseUrlWrap($url)) . ','
    . '\'' . ($newWindow ? md5($url) : 'thePicture') . '\','
    . GeneralUtility::quoteJSvalue('width=' . ($processedFile->getProperty('width') + $offset[0])
    . ',height=' . ($processedFile->getProperty('height') + $offset[1]) . ',status=0,menubar=0')
    . '); return false;';
$a1 = '<a href="' . htmlspecialchars($url) . '"'
    . ' onclick="' . htmlspecialchars($onClick) . '"'
    . ($target !== '' ? ' target="' . htmlspecialchars($target) . '"' : '')
    . $this->getTypoScriptFrontendController()->ATagParams . '>';
$a2 = '</a>';

Some attributes and values are hard coded. Adding custom attributes is not possible.

Actions #12

Updated by Riccardo De Contardi over 5 years ago

@Tobias Schmidt good point. Thank you for your findings

Actions #13

Updated by Gerrit Code Review almost 4 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/c/Packages/TYPO3.CMS/+/63921

Actions #14

Updated by Gerrit Code Review almost 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/+/63921

Actions #15

Updated by Benni Mack almost 4 years ago

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

Updated by Benni Mack almost 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF