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

re-add system contact shortcut #13372

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

fm-sys
Copy link
Contributor

@fm-sys fm-sys commented Jan 20, 2024

Contributor checklist

  • Emulator, Android API 33
  • My contribution is fully baked and ready to be merged as is
  • I ensure that all the open issues my contribution fixes are mentioned in the commit message of my first commit using the Fixes #1234 syntax

Description

Around half a year ago when the conversation fragment was largely rewritten, the long click shortcut to access the system contact was (accidentally?) removed. I thought I would get used to it, but I'm still missing that option... I would really appreciate it if it was readded.

With this PR merged, long-clicking the contact name in the conversation action bar will open the system contact if available.

Copy link

stale bot commented Mar 22, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Mar 22, 2024
@fm-sys
Copy link
Contributor Author

fm-sys commented Mar 22, 2024

Hey @cody-signal @greyson-signal,

Could you be so kind to at least drop off a short message, no matter whether or not this PR gets accepted... Thank you!

Best regards
Florian

@stale stale bot removed the wontfix label Mar 22, 2024
@alex-signal alex-signal reopened this May 9, 2024
@alex-signal
Copy link
Contributor

Hi! I'll run this by the team and make sure we want this functionality reintroduced.

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

Successfully merging this pull request may close these issues.

None yet

2 participants