-
- deleted
-
+
+ deleted
+
diff --git a/src/components/board/SharingTabSidebard.vue b/src/components/board/SharingTabSidebard.vue
index 871a3bc4d..da5e75e36 100644
--- a/src/components/board/SharingTabSidebard.vue
+++ b/src/components/board/SharingTabSidebard.vue
@@ -1,29 +1,29 @@
-
-
-
- {{ scope.option.label }}
-
-
+
+
+
+ {{ scope.option.label }}
+
+
-
- -
-
-
- {{ board.owner.displayname }}
-
-
- -
-
-
- {{ acl.participant.displayname }}
-
-
-
-
+
+ -
+
+
+ {{ board.owner.displayname }}
+
+
+ -
+
+
+ {{ acl.participant.displayname }}
+
+
+
+
diff --git a/src/components/board/TagsTabSidebard.vue b/src/components/board/TagsTabSidebard.vue
index defb2707e..f5d0891ee 100644
--- a/src/components/board/TagsTabSidebard.vue
+++ b/src/components/board/TagsTabSidebard.vue
@@ -4,28 +4,28 @@
-
-
+
+
- {{ label.title }}
+ {{ label.title }}
-
+
-
+
-
-
-
-
+
+
+
+
@@ -41,10 +41,10 @@ import { Compact } from 'vue-color'
export default {
name: 'TagsTabSidebard',
- mixins: [Color],
components: {
'compact-picker': Compact
},
+ mixins: [Color],
data() {
return {
editingLabelId: null,
@@ -58,28 +58,37 @@ export default {
computed: {
...mapGetters({
labels: 'currentBoardLabels'
- }),
+ }),
addLabelObjValidated() {
- if (this.addLabelObj.title == '') return false;
- if (this.addLabelObj.color == '' ||
- this.addLabelObj.color.length != 6) return false;
- return true;
+ if (this.addLabelObj.title === '') {
+ return false
+ }
+
+ if (this.colorIsValid(this.addLabelObj.color) === false) {
+ return false
+ }
+
+ return true
},
editLabelObjValidated() {
- console.log(this.editingLabel.color)
- if (this.editingLabel.title == '') return false;
- if (this.editingLabel.color == '' ||
- this.editingLabel.color.length != 6) return false;
- return true;
+ if (this.editingLabel.title === '') {
+ return false
+ }
+
+ if (this.colorIsValid(this.editingLabel.color) === false) {
+ return false
+ }
+
+ return true
}
-
+
},
methods: {
updateColor(c) {
- if (this.editingLabel == null) {
- this.addLabelObj.color = c.hex.substring(1,7)
+ if (this.editingLabel === null) {
+ this.addLabelObj.color = c.hex.substring(1, 7)
} else {
- this.editingLabel.color = c.hex.substring(1,7)
+ this.editingLabel.color = c.hex.substring(1, 7)
}
},
clickEdit(label) {
@@ -94,8 +103,8 @@ export default {
this.editingLabelId = null
},
clickShowAddLabel() {
- this.addLabelObj = { cardId: null, color: '000', title: ''}
- this.addLabel=true
+ this.addLabelObj = { cardId: null, color: '000', title: '' }
+ this.addLabel = true
},
clickAddLabel() {
this.$store.dispatch('addLabelToCurrentBoard', this.addLabelObj)
diff --git a/src/components/board/TimelineTabSidebard.vue b/src/components/board/TimelineTabSidebard.vue
index cd5548a68..ff9cd5526 100644
--- a/src/components/board/TimelineTabSidebard.vue
+++ b/src/components/board/TimelineTabSidebard.vue
@@ -1,7 +1,7 @@
-
- timeline
-
+
+ timeline
+
diff --git a/src/mixins/color.js b/src/mixins/color.js
index 43d2b5cf2..fbc90580a 100644
--- a/src/mixins/color.js
+++ b/src/mixins/color.js
@@ -81,6 +81,15 @@ export default {
return '#000000'
}
+ },
+ colorIsValid(hex) {
+
+ var re = new RegExp('[A-Fa-f0-9]{6}')
+ if (re.test(hex)) {
+ return true
+ }
+ return false
+
}
}
diff --git a/src/models/index.js b/src/models/index.js
index 4684a3faf..10da0feee 100644
--- a/src/models/index.js
+++ b/src/models/index.js
@@ -52,4 +52,4 @@
* @typedef {Object} Label
* @property {String} title
* @property {String} color
- */
\ No newline at end of file
+ */
diff --git a/src/services/BoardApi.js b/src/services/BoardApi.js
index fbfc699c0..8f8efb23f 100644
--- a/src/services/BoardApi.js
+++ b/src/services/BoardApi.js
@@ -138,47 +138,47 @@ export class BoardApi {
// Label API Calls
deleteLabel(id) {
return axios.delete(this.url(`/labels/${id}`))
- .then(
- (response) => {
- return Promise.resolve(response.data)
- },
- (err) => {
+ .then(
+ (response) => {
+ return Promise.resolve(response.data)
+ },
+ (err) => {
+ return Promise.reject(err)
+ }
+ )
+ .catch((err) => {
return Promise.reject(err)
- }
- )
- .catch((err) => {
- return Promise.reject(err)
- })
+ })
}
updateLabel(label) {
return axios.put(this.url(`/labels/${label.id}`), label)
- .then(
- (response) => {
- return Promise.resolve(response.data)
- },
- (err) => {
+ .then(
+ (response) => {
+ return Promise.resolve(response.data)
+ },
+ (err) => {
+ return Promise.reject(err)
+ }
+ )
+ .catch((err) => {
return Promise.reject(err)
- }
- )
- .catch((err) => {
- return Promise.reject(err)
- })
+ })
}
createLabel(labelData) {
return axios.post(this.url('/labels'), labelData)
- .then(
- (response) => {
- return Promise.resolve(response.data)
- },
- (err) => {
+ .then(
+ (response) => {
+ return Promise.resolve(response.data)
+ },
+ (err) => {
+ return Promise.reject(err)
+ }
+ )
+ .catch((err) => {
return Promise.reject(err)
- }
- )
- .catch((err) => {
- return Promise.reject(err)
- })
+ })
}
}
diff --git a/src/store/main.js b/src/store/main.js
index 0688aa2b1..1cb946e12 100644
--- a/src/store/main.js
+++ b/src/store/main.js
@@ -245,22 +245,22 @@ export default new Vuex.Store({
// label actions
removeLabelFromCurrentBoard({ commit }, label) {
apiClient.deleteLabel(label)
- .then((label) => {
- commit('removeLabelFromCurrentBoard', label.id);
- })
+ .then((label) => {
+ commit('removeLabelFromCurrentBoard', label.id)
+ })
},
updateLabelFromCurrentBoard({ commit }, newLabel) {
apiClient.updateLabel(newLabel)
- .then((newLabel) => {
- commit('updateLabelFromCurrentBoard', newLabel);
- })
+ .then((newLabel) => {
+ commit('updateLabelFromCurrentBoard', newLabel)
+ })
},
addLabelToCurrentBoard({ commit }, newLabel) {
newLabel.boardId = this.state.currentBoard.id
apiClient.createLabel(newLabel)
- .then((newLabel) => {
- commit('addLabelToCurrentBoard', newLabel);
- })
+ .then((newLabel) => {
+ commit('addLabelToCurrentBoard', newLabel)
+ })
}
}
})