Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow user to remove stocks from watchlist. #22

Open
collinpreston opened this issue Mar 27, 2018 · 2 comments
Open

Allow user to remove stocks from watchlist. #22

collinpreston opened this issue Mar 27, 2018 · 2 comments

Comments

@collinpreston
Copy link
Owner

User should be able to long press and have the removal icon in the top right corner of each stock item card display.
@collinpreston
Copy link
Owner Author

This has been implemented. When a long-press is detected on the cardview stock item in any watchlist, a shadow layout is shown over the cardview and a red dumpster icon is shown. If the user selects the red dumpster then the stock item will be removed. Find this is the watchlist fragments for perm, weekly, daily.

@collinpreston
Copy link
Owner Author

This needs to be refactored since price changes cause the cardview to be redrawn which causes issues with displaying the overlay. The idea for refactoring this design is to have a grey dumpster which is shown at all times for every stock item cardview. When the user first presses it, it turns red, then the user presses it again to confirm the deletion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@collinpreston