I've added a menu to share when long-pressing a video #1222
No reviewers
Labels
No labels
android: closed alpha
android: open beta
app-parity
area: devops
area: discovery
area: docs
area: livestream
area: proposal
consider soon
creator
Epic
good first issue
hacktoberfest
help wanted
icebox
Invalid
level: 1
level: 2
level: 3
level: 4
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
on hold
priority: blocker
priority: high
priority: low
priority: medium
product review
resilience
Tom's Wishlist
type: bug
type: discussion
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/lbry-android#1222
Loading…
Reference in a new issue
No description provided.
Delete branch "veritaslogix/09.29.2021-popup_menu"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Hi there!
I'm working on issue #1133 (https://github.com/lbryio/lbry-android/issues/1133)
So far, it's going well. However, there's quite a few ways to implement a menu after a long press. Signal Messenger has a cool UI for when users long click on a message. But other apps like NewPipe just use a dialog to display other options.
To keep things simple, I used a popup menu. I added other menu items that you'd see in the FileViewFragment (download, repost, etc.). I added these because only putting "share" in the menu didn't seem like much of a menu.
(Right now, "Share" is the only menu item that actually works.) Before I keep going, I was looking for some feedback on what I've done so far.
Thanks :)
[PS, I submitted an application to LBRY for an android engineer position on September 9, 2021. This PR is part of my application process.]
PR Checklist
Please check all that apply to this PR using "x":
PR Type
What kind of change does this PR introduce?
Fixes
Issue Number: 1133
What is the current behavior?: Users have to click on a video to use options like sharing
What is the new behavior?: I added a menu so that when users click on a video they can share it without actually clicking on it
Other information
Please, rebase your branch so it is a child from commit ca08f7, latest one you pulled from LBRY-android repository.
By adding a commit to your master branch and then creating a new branch from it, this PR includes that commit, which is unrelated to the feature you are trying to merge to lbry—android.
Then you will need to force-push to your remote. GitHub will likely update automatically this PR.
Okay thanks! So, I've made a new branch to my master branch and rebased this branch to it. Did I understand what you wanted me to do correctly?
Run these commands one by one from master branch:
git checkout 09.29.2021-popup_menu
git rebase --onto
ca08f71
159494And then verify project still builds. Once code has built, run following command, using the id you have setup as your remote (use git remote to see it):
git push -f YOUR_REMOTE_HERE 09.29.2021-popup_menu
This will update your remote branch and GitHub will also show the forced-push and update the commits, now showing only two.
@kekkyojin you are 100% radical. Thank you so much for helping me with that. I just ran those commands they worked perfectly! Thanks!
Please let me know if there's anything you need or if you'd like me to explain my choices in the changes I made.
View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.