beeb / pancaketrade

Limit orders for PancakeSwap
GNU General Public License v3.0
200 stars 91 forks source link

Menu status ordering #5

Closed DigiTecK3D closed 3 years ago

DigiTecK3D commented 3 years ago

This is a small thing, but I went in on my local source copy and switched these menus around since I felt they make more sense this way. You would want to create an order before ever deleting an order and buy or sell before selling all. Just a personal ordering thing. Thank you!

image

beeb commented 3 years ago

Hey! You're right that this ordering makes more sense for the reading order, however as a personal choice, I chose to have the most dangerous operations on the left since I'm right-handed and use my phone a lot. But I understand that it's less logical and there is anyway a confirmation prompt for all operations. I'll think about it!

DigiTecK3D commented 3 years ago

šŸ˜ That does makes sense now when you put it like that, those operations can be dangerous. Then again clicking sell all is pretty dangerous operation to click on also. I havenā€™t run thru any of those operations but I guess just having a ā€œare you sure confirmation on all of itā€ would protect those operations in case someone accidentally clicked them. But thanks for considering it, something I can just change on my end for my preference.