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

Add a submenu with multiple copy options to post menu #1000

Conversation

Astridchan
Copy link

The "Copy text" option in post menu is replaced with a Copy submenu with the following options:

  • Copy link (eg. >>6235023)
  • Copy cross-board link (eg. >>>/g/6235023)
  • Copy post text
  • Copy post URL
  • Copy image URL
  • Copy links in post (opens the "Show links" dialog)
  • Copy post info (opens the "Post info" dialog)

For the last option to make sense, the post info dialog was changed to have selectable text.

In action:

ex1

Other, smaller changes:

  1. Post info now displays the subject and name, allowing the user to copy them to the clipboard.
  2. Swapped position of filter and copy option so copy appears under the main menu and filter under "more..."
  3. "Show links" option is only shown if there is at least one linkable in the post.
  4. Long-press on a link under "Show links" would trigger a normal click and a long click. The fix was to make long click listener return true instead of false.
  5. AlertDialog style was being applied to the context menu causing unwanted effects (see below), Fixed by changing android:background to android:windowBackground.

Screenshot_1595190043

@Adamantcheese Adamantcheese merged commit 89fb815 into Adamantcheese:multi-feature Jul 21, 2020
@Astridchan Astridchan deleted the menu-for-copy-options-in-post-menu branch July 27, 2020 09: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.

2 participants