adhere to convention, prefixing jquery objects with $ and dom not

This commit is contained in:
Tom 2022-05-09 15:15:07 +02:00
parent dc35df9f63
commit 6cae68288d
2 changed files with 6 additions and 6 deletions

View file

@ -187,8 +187,8 @@ async function setContentPane() {
* maxWidth: 100% use full width of container but do not enlarge!
* height:auto to ensure that height scales with width
*/
const svgHtml = $(svg).css({maxWidth: "100%", height: "auto"});
$content.html($('<div>').append(svgHtml));
const $svgHtml = $(svg).css({maxWidth: "100%", height: "auto"});
$content.html($('<div>').append($svgHtml));
} catch(err) {
console.error("error parsing fullNoteRevision.content as JSON", fullNoteRevision.content, err);
$content.html($("<div>").text("Error parsing content. Please check console.error() for more details."));

View file

@ -70,7 +70,7 @@ export default class ExcalidrawTypeWidget extends TypeWidget {
// will be overwritten
this.excalidrawRef;
this.$render;
this.$renderElement;
this.renderElement;
this.$widget;
this.reactHandlers; // used to control react state
@ -114,7 +114,7 @@ export default class ExcalidrawTypeWidget extends TypeWidget {
// this.contentSized();
this.$widget.toggleClass("full-height", true); // only add
this.$render = this.$widget.find('.canvas-note-render');
this.$renderElement = this.$render.get(0);
this.renderElement = this.$render.get(0);
// this.log("doRender", this.$widget);
libraryLoader
@ -125,8 +125,8 @@ export default class ExcalidrawTypeWidget extends TypeWidget {
const React = window.React;
const ReactDOM = window.ReactDOM;
ReactDOM.unmountComponentAtNode(this.$renderElement);
ReactDOM.render(React.createElement(this.ExcalidrawReactApp), this.$renderElement);
ReactDOM.unmountComponentAtNode(this.renderElement);
ReactDOM.render(React.createElement(this.ExcalidrawReactApp), this.renderElement);
})
return this.$widget;