Fix #317: Added multiple layer deletion functionality to action bar

[yashdusing]

Adding multiple layer functionality

[coveralls]

Coverage remained the same at 95.442% when pulling 532bd5625550aa4d1a6a73a60178a8dd94aa9512 on yashdusing:multiple_deletion into c3947686d95c89d643b51a5b5c3be8d93f28476b on Cloud-CV:master.

[Ram81]

@yashdusing can you post a screenshot of how it works?

[yashdusing]

Step 1 : Load model & select the layer you want to delete. Add to the delete queue Step 2 : Repeat for the layers to be deleted Step 3 : Empty the delete queue Output : (Deleting first 3 layers)

[yashdusing]

@thatbrguy @Ram81 could you please review the changes made ?

[yashdusing]

Is this okay?

[thatbrguy]

@yashdusing Hey, I didn't have time to check it out, maybe @Ram81 could.

But from the screenshots, here are my comments:

• The delete button is placed at the right place. Great!
• Is there some highlighter that would indicate what layers are added to the delete queue ?
• Have you tried doing it without a delete queue? As in, delete immediately after selection.

[yashdusing]

I’ll add something to highlight the selected layers

[yashdusing]

Wouldn’t that be the same as the delete button ?

[thatbrguy]

@yashdusing Hey, I tested your method, good work. I would still prefer if it was like the following:

• Press the 'X' button
• Click on any number of layers you want. The click itself either adds the layer to the queue, or deletes it
• Press 'X' again when done

[yashdusing]

I’ll work on the highlighting part first if it’s possible just to make this pr complete and then implement the above . Is that okay ?

[thatbrguy]

Your call. Take your time though, there's no rush.

[yashdusing]

I thought about it for a while. There are 2 approaches : 1: Delete mode button which when activated deletes the layer selected instantly. 2: Delete Queue Add and Empty dropbox which when layer is added to queue, highlight is added to the layer and then selected layers are deleted. What approach do you think is appropriate? The instant delete or highlight ?

[yashdusing]

@thatbrguy @Ram81

[utsavgarg]

@yashdusing Could you fix the PR message, you might have missed deleting the PR guidelines. As for the multiple delete function, could we have a delete option in the sidebar upon clicking which a checkbox pops next to each node, and all layers which have been checked can be deleted on the press of another button.

[yashdusing]

@utsavgarg earlier PR did the same thing (except the checkbox button) you asked. But, I had to shift to the action bar :/.

[utsavgarg]

@yashdusing Sorry about the confusion, by the sidebar I did mean the one on the left (action bar).