Feature Request: Delete multiple layers in Fabrik

[thatbrguy]

As of now, we can delete one layer at a time. We have to select the layer we want to delete and scroll all the way to the bottom to delete the layer.

It's usually a hindrance if you want to make lots of changes to your network, but also do not want to clear everything.

A good solution would be to introduce an option to select and delete multiple layers. Please be sure to update parameter count and other relevant attributes accordingly.

[yashdusing]

Can I work on this ? I'm not well versed with React yet so it might take some time

[thatbrguy]

@yashdusing Verify in Gitter if someone else had already started work before you and if not, take it.

Regarding the issue though, I have a suggestion. You could introduce a button under actions called "Delete mode", and call the already existing delete function whenever delete mode is enabled and you click a layer.

You could also implement your own solution

[Ram81]

@yashdusing I agree with @thatbrguy , doing it with the mode functionality would be better from user's point of view. If you have any other approaches in mind please go ahead with them.

[Shivanshmundra]

@Ram81 @thatbrguy Can you review this?
@thatbrguy Is this what you wanted or something else?

[yashdusing]

I was working on this :/

[Shivanshmundra]

@yashdusing Sorry Due to no activity I thought issue is open.
Never mind, I had just started So you can take up issue.

[yashdusing]

If you don't mind me working on this that is, could you take up another issue or have you implemented it fully?

[Shivanshmundra]

I have just implemented the basic part, So you can look at it and work on this issue.
I will find another issue.

[utsavgarg]

@Shivanshmundra @yashdusing A more intuitive and simpler approach would be to include functionality which allows the user to use the delete key to remove a layer, this would make the process much faster.

[akshitac8]

Is this issue up for working?

[PalashTanejaPro]

@utsavgarg I think this was already implemented in #238

[Gurpreetsingh9465]

Working on this issue

[Ram81]

@Gurpreetsingh9465 sure go ahead.