loleaflet/src/control/Control.JSDialogBuilder.js | 10 +++++----- loleaflet/src/control/Control.Notebookbar.js | 11 ++++++----- 2 files changed, 11 insertions(+), 10 deletions(-)
New commits: commit d0f5989994701f6fe3aadf038798a78dadc8aad5 Author: Szymon Kłos <szymon.k...@collabora.com> AuthorDate: Fri Apr 24 12:44:05 2020 +0200 Commit: Szymon Kłos <szymon.k...@collabora.com> CommitDate: Fri May 1 17:40:04 2020 +0200 jsdialog: identify tabs by id not label Change-Id: Ia695020b5c8afadff8031983c29c527c2ed177dd Reviewed-on: https://gerrit.libreoffice.org/c/online/+/93273 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoff...@gmail.com> Reviewed-by: Szymon Kłos <szymon.k...@collabora.com> diff --git a/loleaflet/src/control/Control.JSDialogBuilder.js b/loleaflet/src/control/Control.JSDialogBuilder.js index 675a084d1..dcb1531bf 100644 --- a/loleaflet/src/control/Control.JSDialogBuilder.js +++ b/loleaflet/src/control/Control.JSDialogBuilder.js @@ -605,7 +605,7 @@ L.Control.JSDialogBuilder = L.Control.extend({ return false; }, - _createTabClick: function(builder, t, tabs, contentDivs, labels) + _createTabClick: function(builder, t, tabs, contentDivs, tabIds) { return function() { $(tabs[t]).addClass('selected'); @@ -617,7 +617,7 @@ L.Control.JSDialogBuilder = L.Control.extend({ } } $(contentDivs[t]).show(); - builder.wizard.selectedTab(labels[t]); + builder.wizard.selectedTab(tabIds[t]); }; }, @@ -629,7 +629,7 @@ L.Control.JSDialogBuilder = L.Control.extend({ var tabs = []; var contentDivs = []; - var labels = []; + var tabIds = []; for (var tabIdx = 0; tabIdx < data.tabs.length; tabIdx++) { var item = data.tabs[tabIdx]; @@ -640,10 +640,10 @@ L.Control.JSDialogBuilder = L.Control.extend({ if (data.selected == data.tabs[tabIdx].id) $(tab).addClass('selected'); tabs[tabIdx] = tab; + tabIds[tabIdx] = tab.id; var label = L.DomUtil.create('span', 'ui-tab-content ' + builder.options.cssClass + ' unolabel', tab); label.innerHTML = title; - labels[tabIdx] = title; var contentDiv = L.DomUtil.create('div', 'ui-content level-' + builder._currentDepth + ' ' + builder.options.cssClass, contentsContainer); contentDiv.title = title; @@ -659,7 +659,7 @@ L.Control.JSDialogBuilder = L.Control.extend({ // to get capture of 't' right has to be a sub fn. var fn = function(id) { return function() { - builder._createTabClick(builder, id, tabs, contentDivs, labels)(); + builder._createTabClick(builder, id, tabs, contentDivs, tabIds)(); builder.callback('tabcontrol', 'selecttab', tabsContainer, id, builder); }; }; diff --git a/loleaflet/src/control/Control.Notebookbar.js b/loleaflet/src/control/Control.Notebookbar.js index ad36e4972..43a680834 100644 --- a/loleaflet/src/control/Control.Notebookbar.js +++ b/loleaflet/src/control/Control.Notebookbar.js @@ -32,24 +32,25 @@ L.Control.Notebookbar = L.Control.extend({ }, selectedTab: function(tabText) { + console.log(tabText); switch (tabText) { - case 'Home': + case 'HomeLabel': this.loadTab(this.getHomeTab()); break; - case 'Insert': + case 'InsertLabel': this.loadTab(this.getInsertTab()); break; - case 'Layout': + case 'LayoutLabel': this.loadTab(this.getLayoutTab()); break; - case 'References': + case 'ReferencesLabel': this.loadTab(this.getReferencesTab()); break; - case 'Table': + case 'TableLabel': this.loadTab(this.getTableTab()); break; } _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits