Fix feedback for markdown toolbar and cke toolbar

pull/1/head
yflory 7 years ago
parent 3fcdbcdbc5
commit 689368cdd7

@ -464,6 +464,7 @@ define([
var AppConfig = common.getAppConfig(); var AppConfig = common.getAppConfig();
var button; var button;
var sframeChan = common.getSframeChannel(); var sframeChan = common.getSframeChannel();
var appType = (common.getMetadataMgr().getMetadata().type || 'pad').toUpperCase();
switch (type) { switch (type) {
case 'export': case 'export':
button = $('<button>', { button = $('<button>', {
@ -656,8 +657,7 @@ define([
case 'toggle': case 'toggle':
button = $('<button>', { button = $('<button>', {
'class': 'fa fa-caret-down cp-toolbar-icon-toggle', 'class': 'fa fa-caret-down cp-toolbar-icon-toggle',
}) });
.click(common.prepareFeedback(type));
window.setTimeout(function () { window.setTimeout(function () {
button.attr('title', data.title); button.attr('title', data.title);
}); });
@ -666,14 +666,16 @@ define([
if (!isVisible) { button.addClass('fa-caret-down'); } if (!isVisible) { button.addClass('fa-caret-down'); }
else { button.addClass('fa-caret-up'); } else { button.addClass('fa-caret-up'); }
}; };
button.click(function () { button.click(function (e) {
data.element.toggle(); data.element.toggle();
var isVisible = data.element.is(':visible'); var isVisible = data.element.is(':visible');
if (callback) { callback(isVisible); } if (callback) { callback(isVisible); }
if (isVisible) { if (isVisible) {
button.addClass('cp-toolbar-button-active'); button.addClass('cp-toolbar-button-active');
if (e.originalEvent) { Feedback.send('TOGGLE_SHOW_' + appType); }
} else { } else {
button.removeClass('cp-toolbar-button-active'); button.removeClass('cp-toolbar-button-active');
if (e.originalEvent) { Feedback.send('TOGGLE_HIDE_' + appType); }
} }
updateIcon(isVisible); updateIcon(isVisible);
}); });
@ -702,7 +704,7 @@ define([
button = $('<button>', { button = $('<button>', {
'class': "fa " + icon, 'class': "fa " + icon,
}) })
.click(common.prepareFeedback(type)); .click(common.prepareFeedback(data.name || 'DEFAULT'));
if (data.title) { button.attr('title', data.title); } if (data.title) { button.attr('title', data.title); }
if (data.style) { button.attr('style', data.style); } if (data.style) { button.attr('style', data.style); }
if (data.id) { button.attr('id', data.id); } if (data.id) { button.attr('id', data.id); }

Loading…
Cancel
Save