Merge pull request #23 from oslabs-beta/keyla/fixDeleteHtmlElementBug

fix html element delete bug, and change the mutations, actions, types

Author: katherinek123

src/components/right-sidebar/HTMLQueue.vue

@@ -144,8 +144,7 @@ export default {
     ...mapActions(['setActiveHTML', 'setActiveLayer', 'upOneLayer', 'setSelectedIdDrag', 'setIdDrag', 'setSelectedIdDrop', 'setIdDrop', 'dragDropSortHtmlElements', 'dragDropSortSelectedHtmlElements', 'openAttributeModal', 'addActiveComponentClass', 'addBindingText']),
     deleteElement(id) {
       if (this.activeComponent === '') this.$store.dispatch(deleteSelectedElement, id[0])
-      this.setActiveHTML(element);
-      this.openAttributeModal(element);
+      else this.$store.dispatch(deleteFromComponentHtmlList, id[1])
 
     },
     setActiveElement(element) {

src/store/actions.js

@@ -246,6 +246,11 @@ const actions = {
   [types.deleteStateFromComponent]: ({ commit }, payload) => {
     commit(types.DELETE_STATE_FROM_COMPONENT, payload);
   },
+  //
+  [types.addBindingText]: ({ commit }, payload) => {
+    commit(types.ADD_BINDING_TEXT, payload)
+  },
+
   // Delete user actions from vuex store
   [types.deleteUserActions]: ({ state, commit }, payload) => {
     commit(types.DELETE_USER_ACTIONS, payload);

src/store/mutations.js

@@ -206,12 +206,7 @@ const mutations = {
       [state.activeComponent]: state.activeComponentObj,
     };
   },
-  // //add binding 
-  [types.addBindingText]: (state, payload) => {
-    //access the htmlList, add payload to the empty bind obj
-    //state.component
-    console.log(state.componentMap[state.activeComponent])
-  },
+  
   [types.DELETE_ACTION_FROM_COMPONENT]: (state, payload) => {
     state.componentMap[state.activeComponent].actions = state.componentMap[state.activeComponent].actions.filter(
       (action) => action !== payload);
@@ -385,7 +380,8 @@ const mutations = {
       text: payload.elementName,
       id: payload.date,
       children: [],
-      class: ''
+      class: '',
+      binding: ''
     });
   },
 
@@ -830,7 +826,7 @@ const mutations = {
   },
 
   // //add binding 
-  [types.addBindingText]: (state, payload) => {
+  [types.ADD_BINDING_TEXT]: (state, payload) => {
     //access the htmlList, add payload to the empty bind obj
     //const active = state.componentMap[state.activeComponent].htmlList;
     if (payload.binding === "") {

src/store/types.js

@@ -85,6 +85,9 @@ export const SET_SELECTED_ID_DROP = 'SET_SELECTED_ID_DROP'
 export const DRAG_DROP_SORT_HTML_ELEMENTS = 'DRAG_DROP_SORT_HTML_ELEMENTS'
 export const DRAG_DROP_SORT_SELECTED_HTML_ELEMENTS = 'DRAG_DROP_SORT_SELECTED_HTML_ELEMENTS'
 
+//adding vmodel info
+export const ADD_BINDING_TEXT = 'ADD_BINDING_TEXT'
+
 // Actions
 export const openNoteModal = 'openNoteModal'
 export const openColorModal = 'openColorModal'