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.

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

Fixes #2388 Replace Toasts with Snackbars #5585

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

neeldoshii
Copy link
Contributor

@neeldoshii neeldoshii commented Mar 2, 2024

Fixes

Purpose/ Description

I have used the methods of Snackbars from ViewUtil and utilized it to convert some toasts to Snackbar.
Note : It doesn't refactor every toast but it covers most of the toasts to snackbar.

Video

Permission Utils Toast (Before)

ca87981c-e06c-4deb-b0a7-e17a0545b06a.mp4

Permission Utils Snackbar (After)

permission_snackbar.webm

Depiction Toast (Before)

depiction.webm

Depiction Snackbar (After)

depiction.mp4

**Recent Search Toast(Before)

recent_search.mp4

**Recent Search Snackbar(After)

recent_search.mp4

@neeldoshii neeldoshii force-pushed the Replace-toasts-with-snackbars-#2388 branch from 158850c to f5f79d2 Compare March 2, 2024 18:28
@neeldoshii neeldoshii marked this pull request as ready for review March 2, 2024 20:33
@neeldoshii neeldoshii force-pushed the Replace-toasts-with-snackbars-#2388 branch from c9f4ec6 to 3ba2aa4 Compare March 8, 2024 15:05
@nicolas-raoul
Copy link
Member

Sorry for the delay in reviewing this!
Would you mind fixing the conflicts and unit tests?
Thanks!

Copy link
Member

@nicolas-raoul nicolas-raoul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Conflict

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace toasts with snackbars where appropriate
2 participants