]> git.koha-community.org Git - koha.git/commit
Bug 12708 - Unexpected behaviour in IE 9 and lower when using openWindow
authorDavid Cook <dcook@prosentient.com.au>
Mon, 4 Aug 2014 03:49:56 +0000 (13:49 +1000)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 19 Aug 2014 12:31:25 +0000 (09:31 -0300)
commit7f8c59edaa9c790f36fe99552e818cfe8d8f9c8f
tree143ebe70feb430fb862ce3187ca2d48f98cbd0c0
parent4daaf8067811ac60b4bca891a5379315b5167127
Bug 12708 - Unexpected behaviour in IE 9 and lower when using openWindow

This patch removes the whitespace from arguments to the "name" parameter
in "window.open". It also adds a trap for the "openWindow" helper
function, which will use a "null" instead of a "name" if window.open
isn't able to handle a name with whitespace (i.e. if it's IE <= 9).

_TEST PLAN_

1) Switch to an emulated or authentic Internet Explorer version E 9
or lower
2) Click "Help"
3) Note that it opens the page in the current window rather than as a popup

4) Apply the patch

5) Shift refresh your page
6) Click "Help"
7) Note that it now opens (correctly) as a pop-up

(If you want to be more thorough, you can test in other browsers as well
to make sure that it still works.)

Tested with IE 10 developer tools: IE7, IE 8, IE 9
Window pops up as expected. No regression found with IE 10 / FF31.0
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested for regressions in Firefox and Chromium.
I checked the help and the duplicate authorities windows
still work correctly.
Passes tests and QA script.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js
koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities.tt
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt