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

streamline help menu #3556

Merged
merged 1 commit into from
Jan 22, 2025
Merged

streamline help menu #3556

merged 1 commit into from
Jan 22, 2025

Conversation

r10s
Copy link
Member

@r10s r10s commented Jan 22, 2025

this PR tweaks the menu atop of the offline help a bit:

  • use clear "Delta Chat Homepage" for opening our homepage (instead of unclear "Learn more about Delta Chat", which is esp. weird, when used from inside the help)

  • do not "promote" GitHub in our menu entry, instead of "Contribute on GitHub", just say "Contribute". destination URL is our contribute page then - that is also more useful for ppl doing eg. translations

came over that when using more modern help menu on ios as a finger exercise at https://github.com/deltachat/deltachat-ios/pull/2545/files :)

this is how it looks like with this PR:

Screenshot 2025-01-22 at 20 43 53

- use clear "Delta Chat Homepage" for opening our homepage
  (instead of unclear "Learn more about Delta Chat",
  which is esp. weird, when used from inside the help)

- do not "promote" GitHub in our menu entry,
  instead of "Contribute on GitHub", just say "Contribute".
  destination URL is our contribute page then -
  that is also more useful for ppl doing eg. translations
@r10s r10s requested a review from adbenitez January 22, 2025 19:51
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@r10s r10s merged commit 3c737f2 into main Jan 22, 2025
2 checks passed
@r10s r10s deleted the r10s/streamline-about-menu branch January 22, 2025 23:51
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.

2 participants