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 Copy Tracker URL on icon long press #1101

Merged
merged 5 commits into from
Aug 11, 2024

Conversation

mm12
Copy link
Contributor

@mm12 mm12 commented Aug 10, 2024

resolves #1087.

When the icon for a tracker is held down, the tracker URL is copied to the clipboard, if it exists.

From @BrutuZ's comment, I have also added it to the menu:
image

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>
@BrutuZ
Copy link

BrutuZ commented Aug 10, 2024

Re: #1087 (comment), how about adding "Copy to clipboard" to the 3-dot menu too?

inb4 Antsy: Switch "Implements" for a supported keyword

@mm12
Copy link
Contributor Author

mm12 commented Aug 10, 2024

Re: #1087 (comment), how about adding "Copy to clipboard" to the 3-dot menu too?

Good idea, I will check into it. In that case, why not also have a copy name item? Or long click the open in browser?

inb4 Antsy: Switch "Implements" for a supported keyword

done :)

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>
@AntsyLich
Copy link
Member

Replace text with "Copy link"?

@mm12
Copy link
Contributor Author

mm12 commented Aug 10, 2024

Replace text with "Copy link"?

I was looking for text that would better suit it, but "Copy to clipboard" is the closest I could find in the locales texts. Should I just add it into there?

@AntsyLich
Copy link
Member

Yeah do that

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>
>
> Co-authored-by: AntsyLich  <59261191+AntsyLich@users.noreply.github.com>

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>
@mm12 mm12 requested a review from AntsyLich August 10, 2024 23:03
@AntsyLich AntsyLich merged commit 200d39e into mihonapp:main Aug 11, 2024
1 check passed
@mm12 mm12 deleted the longPressCopyTrackerLink branch August 11, 2024 18:15
cuong-tran pushed a commit to komikku-app/komikku that referenced this pull request Aug 12, 2024
* Add Copy Tracker URL on icon long press

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>

* Add 'Copy To Clipboard' to tracker item menu

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>

* Add 'Copy link' to locales.

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>

* Implement code review suggestions
>
> Co-authored-by: AntsyLich  <59261191+AntsyLich@users.noreply.github.com>

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>

* Update app/src/main/java/eu/kanade/presentation/track/components/TrackLogoIcon.kt

---------

Signed-off-by: Catt0s <5874051+mm12@users.noreply.github.com>
Co-authored-by: AntsyLich <59261191+AntsyLich@users.noreply.github.com>
(cherry picked from commit 200d39e)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Consistency] Trackers should copy URL to clipboard when icon is long-pressed
3 participants