attempt to fix some bugs with shadows control

This commit is contained in:
Henry Jameson 2020-02-17 23:43:35 +02:00
parent 76323d6d9a
commit 5c6046ea0a
1 changed files with 24 additions and 23 deletions

View File

@ -3,6 +3,17 @@ import OpacityInput from '../opacity_input/opacity_input.vue'
import { getCssShadow } from '../../services/style_setter/style_setter.js' import { getCssShadow } from '../../services/style_setter/style_setter.js'
import { hex2rgb } from '../../services/color_convert/color_convert.js' import { hex2rgb } from '../../services/color_convert/color_convert.js'
const toModel = (object = {}) => ({
x: 0,
y: 0,
blur: 0,
spread: 0,
inset: false,
color: '#000000',
alpha: 1,
...object
})
export default { export default {
// 'Value' and 'Fallback' can be undefined, but if they are // 'Value' and 'Fallback' can be undefined, but if they are
// initially vue won't detect it when they become something else // initially vue won't detect it when they become something else
@ -15,7 +26,7 @@ export default {
return { return {
selectedId: 0, selectedId: 0,
// TODO there are some bugs regarding display of array (it's not getting updated when deleting for some reason) // TODO there are some bugs regarding display of array (it's not getting updated when deleting for some reason)
cValue: this.value || this.fallback || [] cValue: (this.value || this.fallback || []).map(toModel)
} }
}, },
components: { components: {
@ -24,12 +35,12 @@ export default {
}, },
methods: { methods: {
add () { add () {
this.cValue.push(Object.assign({}, this.selected)) this.cValue.push(toModel(this.selected))
this.selectedId = this.cValue.length - 1 this.selectedId = this.cValue.length - 1
}, },
del () { del () {
this.cValue.splice(this.selectedId, 1) this.cValue.splice(this.selectedId, 1)
this.selectedId = this.cValue.length === 0 ? undefined : this.selectedId - 1 this.selectedId = this.cValue.length === 0 ? undefined : Math.max(this.selectedId - 1, 0)
}, },
moveUp () { moveUp () {
const movable = this.cValue.splice(this.selectedId, 1)[0] const movable = this.cValue.splice(this.selectedId, 1)[0]
@ -46,34 +57,24 @@ export default {
this.cValue = this.value || this.fallback this.cValue = this.value || this.fallback
}, },
computed: { computed: {
anyShadows () {
return this.cValue.length > 0
},
anyShadowsFallback () {
return this.fallback.length > 0
},
selected () { selected () {
if (this.ready && this.cValue.length > 0) { if (this.ready && this.anyShadows) {
return this.cValue[this.selectedId] return this.cValue[this.selectedId]
} else { } else {
return { return toModel({})
x: 0,
y: 0,
blur: 0,
spread: 0,
inset: false,
color: '#000000',
alpha: 1
}
} }
}, },
currentFallback () { currentFallback () {
if (this.ready && this.fallback.length > 0) { if (this.ready && this.anyShadowsFallback) {
return this.fallback[this.selectedId] return this.fallback[this.selectedId]
} else { } else {
return { return toModel({})
x: 0,
y: 0,
blur: 0,
spread: 0,
inset: false,
color: '#000000',
alpha: 1
}
} }
}, },
moveUpValid () { moveUpValid () {