barview-android
barview-android copied to clipboard
Being able to drag and adjust the order in which the bars appear in the BarView
Is your feature request related to a problem? Please describe. we arent able to drag and adjust the position of bars in the barView currently Describe the solution you'd like If we long press and then drag we should be able to adjust the position of the bars in the barView
This is a nice idea actually. But it would clash with #54 & #74 as all of these will be based on long clicks. Can you come up with something to avoid that? I'd love to have both the features in the repo π
@krharsh17 I have requested @Alaska18 to use swipe action for deletion since that would be more apt for deletion rather than adjustment π
Well, for her set of features, there has to be a context menu that opens first, and then the user is allowed to choose from a list of options to either delete, fade or highlight a specific bar. Context menus are conventionally put on long-press events, so you'll have to think of a different approach for this.
Suggest me something ,I will also be looking for something that will work!!
Maybe add an option to swap pieces together in @Alaska18 's context menu and then implement this feature with a confirm button at the end of the workflow
Yup, that's a good way to do it. If you can come up with something till tomorrow, we'll consider that tomorrow, or if not, you can begin working on this. @SaumyaSingh1 @ShivangiSingh17 @samagragupta @vikasgola @sidhuparas Your suggestions would be helpful on this π
@krharsh17 something like this maybe having something to drag and adjust rows??
@LuGO0 I was thinking about this drag and drop button just as you shown, but our app is a library so how can we place some kind of Edit button to show these drag buttons?
@sidhuparas what should we do for this reshuffling then any ideas!! Would love to hear em
@LuGO0 ! Thankyou for putting your efforts π―
I really loved your ideaβ€ I think the best way of tackling this is to show a context menu on long press as @krharsh17 suggested β
As said by @krharsh17 -
context menu that opens first, and then the user is allowed to choose from a list of options to either delete, fade or highlight a specific bar
@LuGO0 We might have to reconsider if dragging should be a feature since it might not fit our UI/UX. Do you have some strong reasons to implement this?
@LuGO0 I have a suggestion. When the user long clicks on the bar a menu will open up to select highlight fade or delete and when the user long clicks on label they can move they will he able to drag the bars.
@Alaska18 got your Idea but how do we confirm our reshuffled bars once we are done with adjustments ??
@LuGO0 I can suggest adding a checkbox to the side or at the bottom of whole barview. This checkbox will appear when the user long clicks the label of any bar and when the user is done rearranging the bars, He/she can check the checkbox after which the bars will get rearranged according to the user.
A text can be written next to the checkbox like "okay" or "done".:)
@LuGO0 I have a suggestion. When the user long clicks on the bar a menu will open up to select highlight fade or delete and when the user long clicks on label they can move they will he able to drag the bars.
This sounds good to me ππ» Also, you could consider adding a 're-order' option in the menu to keep things simple and obvious.
@LuGO0 I can suggest adding a checkbox to the side or at the bottom of whole barview. This checkbox will appear when the user long clicks the label of any bar and when the user is done rearranging the bars, He/she can check the checkbox after which the bars will get rearranged according to the user.
For apps that provide a top-level app bar, it'd be better if this confirm button could be integrated in it. Here's a reference on how to achieve that. I am not sure how you can implement it here, but it'd be nice if this could be done. For apps that don't contain an app bar, a button or a checkbox as @Alaska18 suggested can be used.
Thanks everybody for suggestions I will start working on this now !!
@LuGO0 What's the status on this?
Yeah @krharsh17 I was busy with the other PRs now that they are merged I'll continue my work on this :)
@LuGO0 any updates on this?
Sorry @krharsh17 due to my sems resuming I couldnt do much you can unassign me and assign somebody else for this :(