Skip to content

Commit

Permalink
beautify with standard
Browse files Browse the repository at this point in the history
  • Loading branch information
@jfhenon
jfhenon committed Oct 27, 2024
1 parent 346378e commit 594f684
Showing 1 changed file with 127 additions and 55 deletions.
182 changes: 127 additions & 55 deletions packages/svgcanvas/core/draw.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,14 @@ import Layer from './layer.js'
import HistoryRecordingService from './historyrecording.js'

import { NS } from './namespaces.js'
import {
toXml, getElement
} from './utilities.js'
import {
copyElem as utilCopyElem
} from './copy-elem.js'
import { toXml, getElement } from './utilities.js'
import { copyElem as utilCopyElem } from './copy-elem.js'
import { getParentsUntil } from '../common/util.js'

const visElems = 'a,circle,ellipse,foreignObject,g,image,line,path,polygon,polyline,rect,svg,text,tspan,use'.split(',')
const visElems =
'a,circle,ellipse,foreignObject,g,image,line,path,polygon,polyline,rect,svg,text,tspan,use'.split(
','
)

const RandomizeModes = {
LET_DOCUMENT_DECIDE: 0,
Expand Down Expand Up @@ -64,9 +63,10 @@ function isLayerElement (element) {
*/
function getNewLayerName (existingLayerNames) {
let i = 1
// TODO(codedread): What about internationalization of "Layer"?
while (existingLayerNames.includes(('Layer ' + i))) { i++ }
return 'Layer ' + i
while (existingLayerNames.includes(`Layer ${i}`)) {
i++
}
return `Layer ${i}`
}

/**
Expand All @@ -81,9 +81,16 @@ export class Drawing {
* @throws {Error} If not initialized with an SVG element
*/
constructor (svgElem, optIdPrefix) {
if (!svgElem || !svgElem.tagName || !svgElem.namespaceURI ||
svgElem.tagName !== 'svg' || svgElem.namespaceURI !== NS.SVG) {
throw new Error('Error: svgedit.draw.Drawing instance initialized without a <svg> element')
if (
!svgElem ||
!svgElem.tagName ||
!svgElem.namespaceURI ||
svgElem.tagName !== 'svg' ||
svgElem.namespaceURI !== NS.SVG
) {
throw new Error(
'Error: svgedit.draw.Drawing instance initialized without a <svg> element'
)
}

/**
Expand Down Expand Up @@ -260,7 +267,11 @@ export class Drawing {

// if we didn't get a positive number or we already released this number
// then return false.
if (typeof num !== 'number' || num <= 0 || this.releasedNums.includes(num)) {
if (
typeof num !== 'number' ||
num <= 0 ||
this.releasedNums.includes(num)
) {
return false
}

Expand Down Expand Up @@ -353,7 +364,9 @@ export class Drawing {
}

const oldpos = this.indexCurrentLayer()
if ((oldpos === -1) || (oldpos === newpos)) { return null }
if (oldpos === -1 || oldpos === newpos) {
return null
}

// if our new position is below us, we need to insert before the node after newpos
const currentGroup = this.current_layer.getGroup()
Expand Down Expand Up @@ -386,7 +399,9 @@ export class Drawing {
mergeLayer (hrService) {
const currentGroup = this.current_layer.getGroup()
const prevGroup = currentGroup.previousElementSibling
if (!prevGroup) { return }
if (!prevGroup) {
return
}

hrService.startBatchCommand('Merge Layer')

Expand Down Expand Up @@ -490,7 +505,8 @@ export class Drawing {
this.layer_map = {}
const numchildren = this.svgElem_.childNodes.length
// loop through all children of SVG element
const orphans = []; const layernames = []
const orphans = []
const layernames = []
let layer = null
let childgroups = false
for (let i = 0; i < numchildren; ++i) {
Expand Down Expand Up @@ -541,7 +557,12 @@ export class Drawing {
this.current_layer.deactivate()
}
// Check for duplicate name.
if (name === undefined || name === null || name === '' || this.layer_map[name]) {
if (
name === undefined ||
name === null ||
name === '' ||
this.layer_map[name]
) {
name = getNewLayerName(Object.keys(this.layer_map))
}

Expand All @@ -568,10 +589,17 @@ export class Drawing {
* also the current layer of this drawing.
*/
cloneLayer (name, hrService) {
if (!this.current_layer) { return null }
if (!this.current_layer) {
return null
}
this.current_layer.deactivate()
// Check for duplicate name.
if (name === undefined || name === null || name === '' || this.layer_map[name]) {
if (
name === undefined ||
name === null ||
name === '' ||
this.layer_map[name]
) {
name = getNewLayerName(Object.keys(this.layer_map))
}

Expand All @@ -582,8 +610,10 @@ export class Drawing {

// Clone children
const children = [...currentGroup.childNodes]
children.forEach((child) => {
if (child.localName === 'title') { return }
children.forEach(child => {
if (child.localName === 'title') {
return
}
group.append(this.copyElem(child))
})

Expand Down Expand Up @@ -630,7 +660,9 @@ export class Drawing {
return null
}
const layer = this.layer_map[layerName]
if (!layer) { return null }
if (!layer) {
return null
}
layer.setVisible(bVisible)
return layer.getGroup()
}
Expand All @@ -643,7 +675,9 @@ export class Drawing {
*/
getLayerOpacity (layerName) {
const layer = this.layer_map[layerName]
if (!layer) { return null }
if (!layer) {
return null
}
return layer.getOpacity()
}

Expand Down Expand Up @@ -676,7 +710,9 @@ export class Drawing {
*/
copyElem (el) {
const that = this
const getNextIdClosure = function () { return that.getNextId() }
const getNextIdClosure = function () {
return that.getNextId()
}
return utilCopyElem(el, getNextIdClosure)
}
}
Expand All @@ -690,13 +726,20 @@ export class Drawing {
* @returns {void}
*/
export const randomizeIds = function (enableRandomization, currentDrawing) {
randIds = enableRandomization === false
? RandomizeModes.NEVER_RANDOMIZE
: RandomizeModes.ALWAYS_RANDOMIZE

if (randIds === RandomizeModes.ALWAYS_RANDOMIZE && !currentDrawing.getNonce()) {
randIds =
enableRandomization === false
? RandomizeModes.NEVER_RANDOMIZE
: RandomizeModes.ALWAYS_RANDOMIZE

if (
randIds === RandomizeModes.ALWAYS_RANDOMIZE &&
!currentDrawing.getNonce()
) {
currentDrawing.setNonce(Math.floor(Math.random() * 100001))
} else if (randIds === RandomizeModes.NEVER_RANDOMIZE && currentDrawing.getNonce()) {
} else if (
randIds === RandomizeModes.NEVER_RANDOMIZE &&
currentDrawing.getNonce()
) {
currentDrawing.clearNonce()
}
}
Expand Down Expand Up @@ -768,7 +811,7 @@ let svgCanvas
* @param {module:draw.DrawCanvasInit} canvas
* @returns {void}
*/
export const init = (canvas) => {
export const init = canvas => {
svgCanvas = canvas
}

Expand Down Expand Up @@ -802,10 +845,9 @@ export const indexCurrentLayer = () => {
* @returns {void}
*/
export const createLayer = (name, hrService) => {
const newLayer = svgCanvas.getCurrentDrawing().createLayer(
name,
historyRecordingService(hrService)
)
const newLayer = svgCanvas
.getCurrentDrawing()
.createLayer(name, historyRecordingService(hrService))
svgCanvas.clearSelection()
svgCanvas.call('changed', [newLayer])
}
Expand All @@ -822,7 +864,9 @@ export const createLayer = (name, hrService) => {
*/
export const cloneLayer = (name, hrService) => {
// Clone the current layer and make the cloned layer the new current layer
const newLayer = svgCanvas.getCurrentDrawing().cloneLayer(name, historyRecordingService(hrService))
const newLayer = svgCanvas
.getCurrentDrawing()
.cloneLayer(name, historyRecordingService(hrService))

svgCanvas.clearSelection()
leaveContext()
Expand All @@ -845,7 +889,9 @@ export const deleteCurrentLayer = () => {
if (currentLayer) {
const batchCmd = new BatchCommand('Delete Layer')
// store in our Undo History
batchCmd.addSubCommand(new RemoveElementCommand(currentLayer, nextSibling, parent))
batchCmd.addSubCommand(
new RemoveElementCommand(currentLayer, nextSibling, parent)
)
svgCanvas.addCommandToHistory(batchCmd)
svgCanvas.clearSelection()
svgCanvas.call('changed', [parent])
Expand All @@ -861,7 +907,7 @@ export const deleteCurrentLayer = () => {
* @param {string} name - The name of the layer you want to switch to.
* @returns {boolean} true if the current layer was switched, otherwise false
*/
export const setCurrentLayer = (name) => {
export const setCurrentLayer = name => {
const result = svgCanvas.getCurrentDrawing().setCurrentLayer(toXml(name))
if (result) {
svgCanvas.clearSelection()
Expand All @@ -878,11 +924,14 @@ export const setCurrentLayer = (name) => {
* @fires module:svgcanvas.SvgCanvas#event:changed
* @returns {boolean} Whether the rename succeeded
*/
export const renameCurrentLayer = (newName) => {
export const renameCurrentLayer = newName => {
const drawing = svgCanvas.getCurrentDrawing()
const layer = drawing.getCurrentLayer()
if (layer) {
const result = drawing.setCurrentLayerName(newName, historyRecordingService())
const result = drawing.setCurrentLayerName(
newName,
historyRecordingService()
)
if (result) {
svgCanvas.call('changed', [layer])
return true
Expand All @@ -900,12 +949,18 @@ export const renameCurrentLayer = (newName) => {
* 0 and (number of layers - 1)
* @returns {boolean} `true` if the current layer position was changed, `false` otherwise.
*/
export const setCurrentLayerPosition = (newPos) => {
export const setCurrentLayerPosition = newPos => {
const { MoveElementCommand } = svgCanvas.history
const drawing = svgCanvas.getCurrentDrawing()
const result = drawing.setCurrentLayerPosition(newPos)
if (result) {
svgCanvas.addCommandToHistory(new MoveElementCommand(result.currentGroup, result.oldNextSibling, svgCanvas.getSvgContent()))
svgCanvas.addCommandToHistory(
new MoveElementCommand(
result.currentGroup,
result.oldNextSibling,
svgCanvas.getSvgContent()
)
)
return true
}
return false
Expand All @@ -926,7 +981,13 @@ export const setLayerVisibility = (layerName, bVisible) => {
const layer = drawing.setLayerVisibility(layerName, bVisible)
if (layer) {
const oldDisplay = prevVisibility ? 'inline' : 'none'
svgCanvas.addCommandToHistory(new ChangeElementCommand(layer, { display: oldDisplay }, 'Layer Visibility'))
svgCanvas.addCommandToHistory(
new ChangeElementCommand(
layer,
{ display: oldDisplay },
'Layer Visibility'
)
)
} else {
return false
}
Expand All @@ -946,12 +1007,14 @@ export const setLayerVisibility = (layerName, bVisible) => {
* @param {string} layerName - The name of the layer you want to which you want to move the selected elements
* @returns {boolean} Whether the selected elements were moved to the layer.
*/
export const moveSelectedToLayer = (layerName) => {
export const moveSelectedToLayer = layerName => {
const { BatchCommand, MoveElementCommand } = svgCanvas.history
// find the layer
const drawing = svgCanvas.getCurrentDrawing()
const layer = drawing.getLayerByName(layerName)
if (!layer) { return false }
if (!layer) {
return false
}

const batchCmd = new BatchCommand('Move Elements to Layer')

Expand All @@ -960,12 +1023,16 @@ export const moveSelectedToLayer = (layerName) => {
let i = selElems.length
while (i--) {
const elem = selElems[i]
if (!elem) { continue }
if (!elem) {
continue
}
const oldNextSibling = elem.nextSibling
// TODO: this is pretty brittle!
const oldLayer = elem.parentNode
layer.append(elem)
batchCmd.addSubCommand(new MoveElementCommand(elem, oldNextSibling, oldLayer))
batchCmd.addSubCommand(
new MoveElementCommand(elem, oldNextSibling, oldLayer)
)
}

svgCanvas.addCommandToHistory(batchCmd)
Expand All @@ -978,7 +1045,7 @@ export const moveSelectedToLayer = (layerName) => {
* @param {module:history.HistoryRecordingService} hrService
* @returns {void}
*/
export const mergeLayer = (hrService) => {
export const mergeLayer = hrService => {
svgCanvas.getCurrentDrawing().mergeLayer(historyRecordingService(hrService))
svgCanvas.clearSelection()
leaveContext()
Expand All @@ -990,8 +1057,10 @@ export const mergeLayer = (hrService) => {
* @param {module:history.HistoryRecordingService} hrService
* @returns {void}
*/
export const mergeAllLayers = (hrService) => {
svgCanvas.getCurrentDrawing().mergeAllLayers(historyRecordingService(hrService))
export const mergeAllLayers = hrService => {
svgCanvas
.getCurrentDrawing()
.mergeAllLayers(historyRecordingService(hrService))
svgCanvas.clearSelection()
leaveContext()
svgCanvas.changeSvgContent()
Expand Down Expand Up @@ -1032,7 +1101,7 @@ export const leaveContext = () => {
* @fires module:svgcanvas.SvgCanvas#event:contextset
* @returns {void}
*/
export const setContext = (elem) => {
export const setContext = elem => {
const dataStorage = svgCanvas.getDataStorage()
leaveContext()
if (typeof elem === 'string') {
Expand All @@ -1046,9 +1115,12 @@ export const setContext = (elem) => {
const parentsUntil = getParentsUntil(elem, '#svgcontent')
const siblings = []
parentsUntil.forEach(function (parent) {
const elements = Array.prototype.filter.call(parent.parentNode.children, function (child) {
return child !== parent
})
const elements = Array.prototype.filter.call(
parent.parentNode.children,
function (child) {
return child !== parent
}
)
elements.forEach(function (element) {
siblings.push(element)
})
Expand Down

0 comments on commit 594f684

Please sign in to comment.