Skip to content

Conversation

@abhishekbisht1429
Copy link
Contributor

@abhishekbisht1429 abhishekbisht1429 commented Nov 20, 2019

Fixes #1999

Changes:
Replaced all the occurrence of Toast with Snackbar. The class io.pslab.others.CustomSnackbar was used to make Snackbars.

Screenshot/s for the changes:
issue1999_ss1
issue1999_ss2
issue1999_ss3
issue1999_ss4
issue1999_ss5
issue1999_ss6
issue1999_ss7

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing:
issue1999fix.zip

@neel1998
Copy link
Member

Please don't open and close PR for same issue multiple times. Make changes to the same PR. Thank you.

@abhishekbisht1429
Copy link
Contributor Author

Sorry for that! Will be careful hence. Thanks :)

@abhishekbisht1429
Copy link
Contributor Author

Replaced Toast with Snackbar, please have a look and suggest changes if any.

@abhishekbisht1429
Copy link
Contributor Author

abhishekbisht1429 commented Nov 24, 2019

Hi people! Could someone please tell what should i do with this pull request, does it need any changes? I am new to open source contributions and the PSLab project so I am not aware of the usual duration that it takes for a pull request to be reviewed and approved/rejected.

@CloudyPadmal CloudyPadmal added the Enhancement Improvement to an existing feature label Nov 24, 2019
Copy link
Member

@neel1998 neel1998 left a comment

Choose a reason for hiding this comment

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

Looks good. Nice work. Thanks

@neel1998 neel1998 merged commit 702b79a into fossasia:development Nov 26, 2019
@abhishekbisht1429
Copy link
Contributor Author

Thanks and also you are welcome! @neel1998 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Improvement to an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace Toast with Snackbar

4 participants