From fcb9f35dc23eecf28134366500236fc57deb07d4 Mon Sep 17 00:00:00 2001 From: Josh Barnes Date: Fri, 8 Jan 2016 17:53:39 +0000 Subject: [PATCH] in `nbextensions/config`: * fix bug introduced in 8226722c7cab31cfbf49a323770b6c99abb9c8f8, which meant that the activate buttons weren't initialised correctly on loading the extension UI --- nbextensions/config/main.js | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/nbextensions/config/main.js b/nbextensions/config/main.js index 95cc73b..43074cb 100644 --- a/nbextensions/config/main.js +++ b/nbextensions/config/main.js @@ -529,6 +529,12 @@ define([ // for elements that aren't yet part of the DOM ext_ui.css('display', 'none'); ext_ui.insertBefore('.nbext-readme'); + var ext_url = get_ext_url(extension); + var ext_active = false; + if (config.data.hasOwnProperty('load_extensions')) { + ext_active = (config.data.load_extensions[ext_url] === true); + } + set_buttons_active(extension.id, ext_active); } selector.find('li').removeClass('active'); @@ -573,11 +579,11 @@ define([ var state = !$(this).hasClass('nbext-activated'); set_buttons_active(ext_id, state); set_config_active(ext_id, state); + open_ext_ui(a); } - open_ext_ui(a); }; - // fill the columns with nav links, also building UI elements + // fill the columns with nav links var col_length = Math.ceil(extension_list.length / num_cols); for (i in extension_list) { var extension = extension_list[i];