zcy
/
svgedit
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

#se-img-prop-dialog error class set changes

master
Agriya Dev5 2020-12-09 17:14:38 +05:30
parent 375f16e0b2
commit dbe84d8dc7
2 changed files with 14 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/editor/dialogs/imagePropertiesDialog.js
View File

@ -1,5 +1,6 @@
/* eslint-disable node/no-unpublished-import */
import 'elix/define/Dialog.js';
import {isValidUnit} from '../../common/units.js';
const template = document.createElement('template');
template.innerHTML = `
@ -298,6 +299,18 @@ export class SeImgPropDialog extends HTMLElement {
};
const onSaveHandler = (ev) => {
let saveOpt = '';
const w = this.$canvasWidth.value;
const h = this.$canvasHeight.value;
if (w !== 'fit' && !isValidUnit('width', w)) {
this.$canvasWidth.parentElement.classList.add('error');
} else {
this.$canvasWidth.parentElement.classList.remove('error');
}
if (h !== 'fit' && !isValidUnit('height', w)) {
this.$canvasHeight.parentElement.classList.add('error');
} else {
this.$canvasHeight.parentElement.classList.remove('error');
}
if (this.$imageOptEmbed.getAttribute('checked') === 'true') {
saveOpt = 'embed';
}

6
src/editor/svgedit.js
View File

@ -3914,22 +3914,18 @@ editor.init = () => {
*/
const saveDocProperties = function (e) {
// set title
const {title, w, h, save} = e.detail;
const {title, w, h, save} = e.detail;
// set document title
svgCanvas.setDocumentTitle(title);
if (w !== 'fit' && !isValidUnit('width', w)) {
// width.parentElement.classList.add('error');
/* await */ $.alert(uiStrings.notification.invalidAttrValGiven);
return false;
}
// width.parentElement.classList.remove('error');
if (h !== 'fit' && !isValidUnit('height', h)) {
// height.parentElement.classList.add('error');
/* await */ $.alert(uiStrings.notification.invalidAttrValGiven);
return false;
}
// height.parentElement.classList.remove('error');
if (!svgCanvas.setResolution(w, h)) {
/* await */ $.alert(uiStrings.notification.noContentToFitTo);
return false;