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

Added option to clear complete call log. Closes #61. #526

Merged
merged 4 commits into from
Jun 4, 2023
Merged

Added option to clear complete call log. Closes #61. #526

merged 4 commits into from
Jun 4, 2023

Conversation

alvi-khan
Copy link
Contributor

The option to clear all recent calls is added to the 'Settings' menu. I know this isn't the most appropriate place for it but IMO it is the most intuitive.

My proposal for making this better: The fact that tapping on an item from the call log (with 'Group Similar Recents' turned on) brings up a full call log (with recents ungrouped) and not the option to call the number directly is unintuitive. The menu where 'Settings' currently is can be reworked to have three buttons: 'Clear Recents', 'View Complete Call Log' and 'Settings', the third of which opens a new page that shows the settings.

Also, I am aware that there is a bug where, after clearing recents, the 'Are You Sure' menu does not go away by itself. I did not fix this on purpose because the same bug already existed when clearing a single call log and I think the two should be fixed together in a separate issue.

@roeiedri roeiedri changed the base branch from master to staging June 4, 2023 20:12
@roeiedri
Copy link
Collaborator

roeiedri commented Jun 4, 2023

Thank you very much! apologies for the long wait.
Next time just please make the PR to the staging branch :)

@roeiedri roeiedri closed this Jun 4, 2023
@roeiedri roeiedri reopened this Jun 4, 2023
@roeiedri roeiedri merged commit 84b7c79 into Chooloo:staging Jun 4, 2023
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.

None yet

2 participants