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

Don't hide hang up button when showing dialpad (#9) #44

Merged
merged 7 commits into from
Mar 17, 2024

Conversation

Aga-C
Copy link
Member

@Aga-C Aga-C commented Jan 20, 2024

Closes #9

qemu-system-x86_64_DCXKPfPXX3.mp4

@naveensingh
Copy link
Member

@Aga-C The animation doesn't look okay but I don't have any ideas at the moment. I'll think about a better way to do it or you can try again if you want :)

@Aga-C
Copy link
Member Author

Aga-C commented Jan 20, 2024

What exactly could be done better?

@inson1
Copy link

inson1 commented Jan 20, 2024

I think it looks good. Even if there could be better way, this is better than nothing, so If some idea doesnt come, just merge it before next release, please

@Aga-C But what about bigger space between, the button and dialpad? just little bit?

@Aga-C
Copy link
Member Author

Aga-C commented Jan 20, 2024

@inson1 It's the same margin as in dialpad (the one before calling).

@inson1
Copy link

inson1 commented Jan 20, 2024

ok (that could be bigger but that isnt part of this PR)

@naveensingh
Copy link
Member

Based on your recording, during the dialpad animation, the hangup button is below the semi-transparent dialpad and that looks like a glitch instead of an animation.

Check this old Google Phone APK for inspiration.

Without redesigning the UI, we can do one of these:

  • Elevate the hangup button such that the dialpad appears behind it but in front of the other call UI buttons. In this case, the hangup button should always have some android:elevation set.
  • Hide the hangup button using shrink animation as it is currently -> Show the dialpad -> Show the hangup button using scale animation.

In any case, we should increase the dialpad bottom padding slightly so there's more padding between the hangup button and the 0 button on the dialpad.

@naveensingh
Copy link
Member

Even if there could be better way, this is better than nothing, so If some idea doesnt come, just merge it before next release

Something is always better than nothing, I absolutely agree with that. But in general and regarding any current or future contributions, I'll never merge a PR if I think the problem could have been solved in a better way. That is to enforce quality contributions and for my own peace of mind :)

@Aga-C
Copy link
Member Author

Aga-C commented Jan 20, 2024

I've added some margin to the bottom of the dialpad and went with the first option of changing the animation (added elevation).

@naveensingh naveensingh merged commit ef7aadc into FossifyOrg:master Mar 17, 2024
@naveensingh
Copy link
Member

Just made some UI changes to enable the dialpad elevation, thanks 👍

@Aga-C Aga-C deleted the dont-hide-hang-up-button branch March 17, 2024 08:34
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.

Keep "Hang Up" button on dialer
3 participants