enviroCar / enviroCar-app

enviroCar Android Application
https://envirocar.org
GNU General Public License v3.0
88 stars 154 forks source link

Add bin icon to delete car in my cars scree #448

Open neel1998 opened 4 years ago

neel1998 commented 4 years ago

Currenty the delete option appears when user clicks on the car. Which is not very much apparent for the first time user. Having a bin icon would make it easy

Current behavior 20200221_022457

I would like to work on this issue

asaikarthikeya commented 3 years ago

Working on this.

cdhiraj40 commented 2 years ago

@asaikarthikeya what exactly mean by "having a bin icon" ? adding it around the radiobutton? if the issue/feature is still relevant I would like to try :)

asaikarthikeya commented 2 years ago

@asaikarthikeya what exactly mean by "having a bin icon" ? adding it around the radiobutton? if the issue/feature is still relevant I would like to try :)

If you see OBD selection screen, you can see that for every paired device it has a bin symbol(to delete)and a select option. Similar thing is expected in car selection screen

Let's wait to hear @SebaDro views on this issue. If sir feels this feature is good to add, you can work on this. Thanks 🙂

cdhiraj40 commented 2 years ago

@asaikarthikeya what exactly mean by "having a bin icon" ? adding it around the radiobutton? if the issue/feature is still relevant I would like to try :)

If you see OBD selection screen, you can see that for every paired device it has a bin symbol(to delete)and a select option. Similar thing is expected in car selection screen

Let's wait to hear @SebaDro views on this issue. If sir feels this feature is good to add, you can work on this. Thanks

sure I will wait :)

asaikarthikeya commented 2 years ago

Definitely, bin icon provides a better ui experience but not really sure if this is required in our app because, some car models have really long name, such car names are not displayed completely. This is due to the presence of car symbol on the left side. By adding a bin symbol on the right end the car name will be further be reduced in size. May be only the car manufacturer name will be visible. In my opinion this is may not suit in our app ui.

@SebaDro is this required sir? What are your thoughts on this? Thanks

cdhiraj40 commented 2 years ago

yes, I totally agree if we don't show car names in the car list then I guess it doesn't make sense. if @SebaDro would like this feature then I will sure try to work on it else I will try doing something else. Thanks

Definitely, bin icon provides a better ui experience but not really sure if this is required in our app because, some car models have really long name, such car names are not displayed completely. This is due to the presence of car symbol on the left side. By adding a bin symbol on the right end the car name will be further be reduced in size. May be only the car manufacturer name will be visible. In my opinion this is may not suit in our app ui.

@SebaDro is this required sir? What are your thoughts on this? Thanks

yes, I totally agree if we don't show car names properly in the car list then I guess it doesn't make sense. if @SebaDro would like this feature then I will sure try to work on it else I will try doing something else. Thanks

SebaDro commented 2 years ago

The current workflow for deleting an entry is quite sufficient. An extra bin icon would be only nice to have, so priority for this enhancement is very low. However, if you come up with a good design, that also takes into account long device names, I would consider to merge it.

pree-T commented 2 years ago

May I work on it?

SanjaySargam commented 1 year ago

Can I work on this issue?