Minor refactoring...
This commit is contained in:
parent
7b556a676c
commit
aa6bae2a24
11
js/scene.js
11
js/scene.js
@ -184,22 +184,21 @@ Scene.toggleSettingsPanel = function(triggerId) {
|
|||||||
* @param {String} triggerSelector
|
* @param {String} triggerSelector
|
||||||
*/
|
*/
|
||||||
Scene.UI.toggleClipper = function(triggerSelector) {
|
Scene.UI.toggleClipper = function(triggerSelector) {
|
||||||
document.querySelector(triggerSelector).addEventListener(
|
const trigger = document.querySelector(triggerSelector);
|
||||||
|
trigger.addEventListener(
|
||||||
'click',
|
'click',
|
||||||
() => {
|
() => {
|
||||||
if (!AppState.clipping.enabled) {
|
if (!AppState.clipping.enabled) {
|
||||||
AppState.clipping.enabled = true;
|
AppState.clipping.enabled = true;
|
||||||
document.querySelector(triggerSelector).className += ' border border-2 border-info';
|
trigger.className += ' border border-2 border-white';
|
||||||
//Scene.activateClipper()
|
//Scene.activateClipper()
|
||||||
} else {
|
} else {
|
||||||
AppState.clipping.enabled = false;
|
AppState.clipping.enabled = false;
|
||||||
ATON.disableClipPlanes();
|
ATON.disableClipPlanes();
|
||||||
AppState.root.remove(AppState.clipping.helper);
|
AppState.root.remove(AppState.clipping.helper);
|
||||||
AppState.clipping.helper = null;
|
AppState.clipping.helper = null;
|
||||||
// Refactor!!
|
let noBorder = trigger.className.replace(/ border.*$/g, '');
|
||||||
document.querySelector(triggerSelector).classList.remove('border-info');
|
trigger.className = noBorder;
|
||||||
document.querySelector(triggerSelector).classList.remove('border-2');
|
|
||||||
document.querySelector(triggerSelector).classList.remove('border');
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
);
|
);
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user