From 9826ecd5c3b79320823cd762880c302253e88d06 Mon Sep 17 00:00:00 2001
From: Simon Tatham <anakin@pobox.com>
Date: Sun, 31 Mar 2013 09:58:46 +0000
Subject: Apply a bodge to arrange that if the user selects Custom from the
 game type dropdown, we still get an 'onchange' event if they select it a
 second time. Normally this wouldn't happen, because onchange means what it
 says and we only get it if a _different_ element is selected.

My solution is to create two list items called Custom, set one of them
as display:none to stop it showing up when the list is dropped down,
and to select it after the configuration box closes.

[originally from svn r9788]
---
 emccpre.js | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'emccpre.js')

diff --git a/emccpre.js b/emccpre.js
index 548e2a5..ea20771 100644
--- a/emccpre.js
+++ b/emccpre.js
@@ -78,7 +78,14 @@ var dlg_return_sval, dlg_return_ival;
 // The <select> object implementing the game-type drop-down, and a
 // list of the <option> objects inside it. Used by js_add_preset(),
 // js_get_selected_preset() and js_select_preset().
+//
+// gametypehiddencustom is a second copy of the 'Custom' dropdown
+// element, set to display:none. This is used by a bodge in emcclib.js
+// (see comment in js_add_preset) to arrange that if the Custom
+// element is (apparently) already selected, we still find out if the
+// user selects it again.
 var gametypeselector = null, gametypeoptions = [];
+var gametypehiddencustom = null;
 
 // The two anchors used to give permalinks to the current puzzle. Used
 // by js_update_permalinks().
-- 
cgit v1.1