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

Closes issue #1310 : Adds Select All Conversations Feature #1652

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

goforbg
Copy link

@goforbg goforbg commented Oct 2, 2020

  • Shows a "Select All" feature in the main menu bar to Mark all as read / Unread, Pin to Top, Block All and Archive All.

  • Hides "Select All" when the list is empty / when all items are already selected manually (or using the Select All button)

  • Added translations for all languages

  • Provided comments on why it's implemented at child adapter-level.

This is my first PR at this repo, first time working with MVP architecture, would love to contribute more after feedback 🎉

@GeoZac
Copy link
Contributor

GeoZac commented Oct 2, 2020

Is it possible to split the PR into individual commits, the checklist is the good way to split the commits. This will help in debugging any issues

@goforbg
Copy link
Author

goforbg commented Oct 2, 2020

Sure will do that.

@goforbg
Copy link
Author

goforbg commented Oct 2, 2020

@GeoZac Can you check now? Split it into separate commits.

@goforbg
Copy link
Author

goforbg commented Oct 6, 2020

is @moezbhatti the only maintainer?

@cleanerspam
Copy link

Will this also add option to multi select receivers from contact list ?
Like long tapping any one contact presents a check box on all contacts and then you can select multiple contacts

@goforbg
Copy link
Author

goforbg commented Jan 6, 2021

@cleanerspam nope, just adds an option to the menu.

@iamtalhaasghar
Copy link

Please merge this request. We need this feature.
Thanks @goforbg for your efforts.

@TechnologyClassroom
Copy link

Link to issue #1789

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.

5 participants