[BUGFIX] EM: Popup window not scrollable
authorStefan Galinski <stefan.galinski@gmail.com>
Fri, 21 Oct 2011 22:44:33 +0000 (00:44 +0200)
committerXavier Perseguers <xavier@typo3.org>
Sat, 22 Oct 2011 09:50:27 +0000 (11:50 +0200)
There are four places in the Extension Manager, when a JavaScript popup
is opened (not an ExtJS window).

These popups are created with
  window.open(..., width=650,height=500),
thus they have no scrollbars. While it is possible in most browsers to
scroll with a mouse wheel, this seems not to be possible with IE.

Change-Id: I71ed8393c24bc2a6e543b1e198ceb1bf240f1522
Resolves: #31184
Releases: 4.6, 4.5
Reviewed-on: http://review.typo3.org/6185
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
typo3/sysext/em/classes/install/class.tx_em_install.php

index f3dc963..ddb4288 100644 (file)
@@ -497,7 +497,7 @@ class tx_em_Install {
                                                        'CMD[importExt]' => $depK,
                                                        'CMD[loc]' => 'L',
                                                        'CMD[standAlone]' => 1
-                                               )) . '\', \'' . md5($instExtInfo[$depK]['EM_CONF']['title']) . '\', \'width=650, height=500\'); return false;" target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_import_now') . '</a>';
+                                               )) . '\', \'' . md5($instExtInfo[$depK]['EM_CONF']['title']) . '\', \'width=650,height=500,scrollbars=yes\'); return false;" target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_import_now') . '</a>';
                                                $msg[] = '&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" value="1" name="depsolver[ignore][' . $depK . ']" id="checkIgnore_' . $depK . '" />
                                                        <label for="checkIgnore_' . $depK . '">' . $GLOBALS['LANG']->getLL('checkDependencies_ignore_ext_requirement') . '</label>';
                                        } else {
@@ -511,7 +511,7 @@ class tx_em_Install {
                                                        'CMD[standAlone]' => 1,
                                                        'SET[singleDetails]' => 'info'
                                                )) .
-                                                               '\', \'' . md5($instExtInfo[$depK]['EM_CONF']['title']) . '\', \'width=650, height=500\'); return false; " target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_install_now') . '</a>';
+                                                               '\', \'' . md5($instExtInfo[$depK]['EM_CONF']['title']) . '\', \'width=650,height=500,scrollbars=yes\'); return false; " target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_install_now') . '</a>';
                                                $msg[] = '&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" value="1" name="depsolver[ignore][' . $depK . ']" id="checkIgnore_' . $depK . '" />
                                                        <label for="checkIgnore_' . $depK . '">' . $GLOBALS['LANG']->getLL('checkDependencies_ignore_ext_requirement') . '</label>';
                                        }
@@ -611,7 +611,7 @@ class tx_em_Install {
                                                        'CMD[importExt]' => $suggestK,
                                                        'CMD[loc]' => 'L',
                                                        'CMD[standAlone]' => 1
-                                               )) . '\', \'' . md5($suggestK) . '\', \'width=650, height=500\'); return false; " target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_import_now') . '</a>';
+                                               )) . '\', \'' . md5($suggestK) . '\', \'width=650,height=500,scrollbars=yes\'); return false; " target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_import_now') . '</a>';
                                                $msg[] = '&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" value="1" name="depsolver[ignore][' . $suggestK . ']" id="checkIgnore_' . $suggestK . '" />
                                                        <label for="checkIgnore_' . $suggestK . '">' . $GLOBALS['LANG']->getLL('checkDependencies_ignore_suggestion') . '</label>';
                                        } else {
@@ -625,7 +625,7 @@ class tx_em_Install {
                                                        'CMD[standAlone]' => 1,
                                                        'SET[singleDetails]' => 'info'
                                                ))) .
-                                                               '\', \'' . md5($suggestK) . '\', \'width=650, height=500\'); return false; " target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_install_now') . '</a>';
+                                                               '\', \'' . md5($suggestK) . '\', \'width=650,height=500,scrollbars=yes\'); return false; " target="_blank">' . $GLOBALS['LANG']->getLL('checkDependencies_install_now') . '</a>';
                                                $msg[] = '&nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" value="1" name="depsolver[ignore][' . $suggestK . ']" id="checkIgnore_' . $suggestK . '" />
                                                        <label for="checkIgnore_' . $suggestK . '">' . $GLOBALS['LANG']->getLL('checkDependencies_ignore_suggestion') . '</label>';
                                        }