search

kapieyow / pickem

2 stars 4 forks source link

Confirm changing pick if one already selected #54

Open Esuoh opened 4 years ago

Esuoh commented 4 years ago

To avoid accidental changing of picks when scrolling, present a prompt to change a pick if one has already been made for a match-up.

kapieyow commented 4 years ago

I have considered this at points but never followed through. I couldn't figure out how to make it clean without being too intrusive in "normal pick change" cases. Will be cool to see what you come up with.

The UI code is probably near here

https://github.com/kapieyow/pickem/blob/35b02216e3b2b88052bf9a16ebd3d6982fb975c7/pickem-web/src/app/player/player.component.ts#L32

There is an example of a bootstrap dialog here, but it may be way overkill for what you are trying to do.

https://github.com/kapieyow/pickem/blob/35b02216e3b2b88052bf9a16ebd3d6982fb975c7/pickem-web/src/app/top-nav/top-nav.component.html#L62

Very interested to see what you come up with. Happy to host on a test site or whatever. Let me know if you have questions.