From 4e1675fa47595979321f6ae8b7588bc61f24ff79 Mon Sep 17 00:00:00 2001 From: Mike Taylor Date: Fri, 17 Oct 2014 15:38:03 +0100 Subject: [PATCH] Revert "Towards MKWS-294 (Change names of MKWS classes to be case-insensitive)" I never intended to make these changes in master, but in new-class-names. This reverts commit f944d2112f977ffe31545ceaee8a1ebeb1e128d8. --- src/mkws-core.js | 32 ++++--------- src/mkws-popup.js | 6 +-- src/mkws-team.js | 16 +++---- src/mkws-widget-authname.js | 2 +- src/mkws-widget-builder.js | 14 +++--- src/mkws-widget-categories.js | 4 +- src/mkws-widget-log.js | 2 +- src/mkws-widget-main.js | 50 ++++++++++---------- src/mkws-widget-record.js | 18 +++---- src/mkws-widget-reference.js | 4 +- .../Reference.handlebars | 2 +- src/mkws-widget-termlists.js | 4 +- src/mkws-widget.js | 4 +- src/mkws.templates/Facet.handlebars | 4 +- src/mkws.templates/Navi.handlebars | 2 +- src/mkws.templates/Pager.handlebars | 10 ++-- src/mkws.templates/Progress.handlebars | 4 +- src/mkws.templates/Ranking.handlebars | 4 +- src/mkws.templates/Results.handlebars | 16 +++---- src/mkws.templates/Search.handlebars | 6 +-- src/mkws.templates/Stat.handlebars | 2 +- src/mkws.templates/Termlists.handlebars | 4 +- 22 files changed, 97 insertions(+), 113 deletions(-) diff --git a/src/mkws-core.js b/src/mkws-core.js index df7382f..3a4fa5b 100644 --- a/src/mkws-core.js +++ b/src/mkws-core.js @@ -286,16 +286,6 @@ mkws.log("Using window.name '" + window.name + "'"); // wrapper to provide local copy of the jQuery object. (function($) { var log = mkws.log; - var _old2new = { // Maps old-style widget names to new-style - 'Authname': 'auth-name', - 'ConsoleBuilder': 'console-builder', - 'Coverart': 'cover-art', - 'GoogleImage': 'google-image', - 'MOTD': 'motd', - 'MOTDContainer': 'motd-container', - 'Perpage': 'per-page', - 'SearchForm': 'search-form', - }; function handleNodeWithTeam(node, callback) { // First branch for DOM objects; second branch for jQuery objects @@ -313,15 +303,10 @@ mkws.log("Using window.name '" + window.name + "'"); for (var i = 0; i < list.length; i++) { var cname = list[i]; - if (cname.match(/^mkws-team-/)) { - teamName = cname.replace(/^mkws-team-/, ''); - } else if (cname.match(/^mkws-/)) { - // New-style names of the form mkws-foo-bar - type = cname.replace(/^mkws-/, ''); + if (cname.match(/^mkwsTeam_/)) { + teamName = cname.replace(/^mkwsTeam_/, ''); } else if (cname.match(/^mkws/)) { - // Old-style names of the form mkwsFooBar - var tmp = cname.replace(/^mkws/, ''); - type = _old2new[tmp] || tmp; + type = cname.replace(/^mkws/, ''); } } @@ -368,8 +353,8 @@ mkws.log("Using window.name '" + window.name + "'"); for (var tname in mkws.teams) { var team = mkws.teams[tname]; team.visitWidgets(function(t, w) { - var w1 = team.widget(t + "-container-" + from); - var w2 = team.widget(t + "-container-" + to); + var w1 = team.widget(t + "-Container-" + from); + var w2 = team.widget(t + "-Container-" + to); if (w1) { w1.node.hide(); } @@ -458,10 +443,9 @@ mkws.log("Using window.name '" + window.name + "'"); var s = ""; for (var type in mkws.widgetType2function) { if (s) s += ','; - s += '.mkws-' + type; - s += ',.mkws-' + type + "-container-wide"; - s += ',.mkws-' + type + "-container-narrow"; - // ### Do we need to do something about old-style names? + s += '.mkws' + type; + s += ',.mkws' + type + "-Container-wide"; + s += ',.mkws' + type + "-Container-narrow"; } return s; } diff --git a/src/mkws-popup.js b/src/mkws-popup.js index ee26049..7496a07 100644 --- a/src/mkws-popup.js +++ b/src/mkws-popup.js @@ -4,13 +4,13 @@ //"use strict"; // $(document).ready(function () { -mkws.registerWidgetType('popup', function() { +mkws.registerWidgetType('Popup', function() { var $ = mkws.$; var debug = mkws.log; debug("init popup window"); var popup_window = $(this.node); - // var popup_window = $(".mkws-popup"); // $(document).ready() + // var popup_window = $(".mkwsPopup"); // $(document).ready() if (!popup_window) { debug("no popup found, skip..."); return; @@ -49,7 +49,7 @@ mkws.registerWidgetType('popup', function() { close: function() {} }); - // open at search query submit: "input.mkws-button" + // open at search query submit: "input.mkwsButton" var id_botton = that.attr("popup_button"); if (id_botton) { $(id_botton).button().click(function() { diff --git a/src/mkws-team.js b/src/mkws-team.js index 0825553..0879add 100644 --- a/src/mkws-team.js +++ b/src/mkws-team.js @@ -191,13 +191,13 @@ mkws.makeTeam = function($, teamName) { // Used by the Records widget and onRecord() function recordElementId(s) { - return 'mkws-rec_' + s.replace(/[^a-z0-9]/ig, '_'); + return 'mkwsRec_' + s.replace(/[^a-z0-9]/ig, '_'); } that.recordElementId = recordElementId; // Used by onRecord(), showDetails() and renderDetails() function recordDetailsId(s) { - return 'mkws-det_' + s.replace(/[^a-z0-9]/ig, '_'); + return 'mkwsDet_' + s.replace(/[^a-z0-9]/ig, '_'); } @@ -390,10 +390,10 @@ mkws.makeTeam = function($, teamName) { teamName = teamName || m_teamName; if (teamName === 'AUTO') { - selector = (selector + '.mkws-team-' + teamName + ',' + - selector + ':not([class^="mkwsTeam"],[class*=" mkwsTeam"],[class^="mkws-team-"],[class*=" mkws-team-"])'); + selector = (selector + '.mkwsTeam_' + teamName + ',' + + selector + ':not([class^="mkwsTeam"],[class*=" mkwsTeam"])'); } else { - selector = selector + '.mkws-team-' + teamName; + selector = selector + '.mkwsTeam_' + teamName; } var node = $(selector); @@ -410,7 +410,7 @@ mkws.makeTeam = function($, teamName) { function renderDetails(data, marker) { var template = loadTemplate("Record"); var details = template(data); - return '
' + details + '
'; } that.renderDetails = renderDetails; @@ -425,9 +425,9 @@ mkws.makeTeam = function($, teamName) { var template = m_template[name]; if (template === undefined && Handlebars.compile) { var source; - var node = $(".mkws-template_" + name + " .mkws-team-" + that.name()); + var node = $(".mkwsTemplate_" + name + " .mkwsTeam_" + that.name()); if (node && node.length < 1) { - node = $(".mkws-template_" + name); + node = $(".mkwsTemplate_" + name); } if (node) source = node.html(); if (!source) source = m_templateText[name]; diff --git a/src/mkws-widget-authname.js b/src/mkws-widget-authname.js index b96dad2..b1e0b1f 100644 --- a/src/mkws-widget-authname.js +++ b/src/mkws-widget-authname.js @@ -1,4 +1,4 @@ -mkws.registerWidgetType('auth-name', function() { +mkws.registerWidgetType('Authname', function() { var that = this; this.team.queue("authenticated").subscribe(function(authName) { diff --git a/src/mkws-widget-builder.js b/src/mkws-widget-builder.js index dac54ab..6468e86 100644 --- a/src/mkws-widget-builder.js +++ b/src/mkws-widget-builder.js @@ -1,4 +1,4 @@ -mkws.registerWidgetType('builder', function() { +mkws.registerWidgetType('Builder', function() { var that = this; var team = this.team; @@ -8,11 +8,11 @@ mkws.registerWidgetType('builder', function() { }); this.node.append(button); button.click(function() { - var query = team.widget('query').value(); - var sort = team.widget('sort').value(); - var perpage = team.widget('per-page').value(); + var query = team.widget('Query').value(); + var sort = team.widget('Sort').value(); + var perpage = team.widget('Perpage').value(); - var html = ('
'); @@ -21,8 +21,8 @@ mkws.registerWidgetType('builder', function() { }); }); -mkws.registerWidgetType('console-builder', function() { - mkws.promotionFunction('builder').call(this); +mkws.registerWidgetType('ConsoleBuilder', function() { + mkws.promotionFunction('Builder').call(this); this.callback = function(s) { console.log("generated widget: " + s); } diff --git a/src/mkws-widget-categories.js b/src/mkws-widget-categories.js index 754e68b..6982db8 100644 --- a/src/mkws-widget-categories.js +++ b/src/mkws-widget-categories.js @@ -1,4 +1,4 @@ -mkws.registerWidgetType('categories', function() { +mkws.registerWidgetType('Categories', function() { var that = this; if (!mkws.authenticated) { @@ -20,7 +20,7 @@ mkws.registerWidgetType('categories', function() { var text = []; text.push("Select category: "); - text.push(" + + - +
+ +
diff --git a/src/mkws.templates/Stat.handlebars b/src/mkws.templates/Stat.handlebars index c912c22..d8ae997 100644 --- a/src/mkws.templates/Stat.handlebars +++ b/src/mkws.templates/Stat.handlebars @@ -6,4 +6,4 @@ clients - total targets for this search records - number of records returned and available hits - number of hits across all targets }} - -- {{{mkws-translate "Active clients"}}} : {{activeclients}}/{{clients}} -- {{{mkws-translate "Retrieved records"}}} : {{records}}/{{hits}} + -- {{{mkws-translate "Active clients"}}} : {{activeclients}}/{{clients}} -- {{{mkws-translate "Retrieved records"}}} : {{records}}/{{hits}} diff --git a/src/mkws.templates/Termlists.handlebars b/src/mkws.templates/Termlists.handlebars index 5ee6f6b..7c6f3b8 100644 --- a/src/mkws.templates/Termlists.handlebars +++ b/src/mkws.templates/Termlists.handlebars @@ -5,7 +5,7 @@ team - the current team facets - array of facet names }} -
Termlists
+
Termlists
{{#each facets}} -
+
{{/each}} -- 1.7.10.4