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.

Sign up for GitHub

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

Jump to bottom

[Feature] Enable copy to clipboard for notifications #224

Closed
clsty opened this issue Feb 1, 2024 · 2 comments
Closed

[Feature] Enable copy to clipboard for notifications #224

clsty opened this issue Feb 1, 2024 · 2 comments

Comments

@clsty
Copy link
Collaborator

clsty commented Feb 1, 2024 •
edited
Loading

What would you like to be added?
Enable copy content of notification to clipboard.

  • Maybe add a "Copy" button on each notification.
  • Or, enable selection on the text (so that user can copy them by pressing Ctrl+C or Ctrl+Shift+C).

How will it help
E.g. when such notification pops up like #222 (comment) , user will be able to copy it, not screenshot or take a picture of it.
end-4 added a commit that referenced this issue Feb 2, 2024
@end-4
hold notif to copy (#224)
8589654
@end-4
Copy link
Owner

end-4 commented Feb 2, 2024 •
edited
Loading

hmm

  • I don't like an extra button since imo there are already enough
  • Selection won't work because there are gestures
    • Then I thought: gestures -> hmm mobile -> copying on mobile? -> touch and hold!

... So now if you click and hold a notif for 800ms its content will be copied

video

recording_20240202_19-25-18.mp4

@clsty
Copy link
Collaborator Author

clsty commented Feb 2, 2024

Then I thought: gestures -> hmm mobile -> copying on mobile? -> touch and hold!

That's a nice thought.

I've tried it. It's a very cool design! Thanks!

@clsty clsty closed this as completed Feb 2, 2024
czlabinger added a commit to mkurz-TGM/dots that referenced this issue Feb 3, 2024
@czlabinger @loeken
@end-4 @clsty
Merge (#7)
face658 
* Update system.js

* Update music.js

* minor ai changes

* update description (end-4#229)

* fix sddm L (end-4#228)

* Added rust lsp, moven weather into seperate const to use on laptop as well

* format

* hold notif to copy (end-4#224)

* overview: fix bug for multimonitor (end-4#212, end-4#221)

* FIX SPAMMING NOTIF ERRORS FINALLY OMGGGGGGGGGG (btw end-4#215)

* fix timeout anim (hmm end-4#215)

* re-add notif force dismissal (end-4#215)

* hmm

* overview: fix row 1 not showing when ws 6 is occupied

* add python-setuptools-scm, python-wheel (end-4#197)

* starship: shorter dir display

* Implement bash-style comment for dependencies list

* Minor fixes.

* ags: sync

* fix CreateCtor error for newer ags versions

* update ags

* end-4#233

* Nvim

---------

Co-authored-by: loeken <loeken@internetz.me>
Co-authored-by: end-4 <97237370+end-4@users.noreply.github.com>
Co-authored-by: clsty <celestial.y@outlook.com>
end-4 added a commit that referenced this issue Feb 22, 2024
@end-4
hold notif to copy (#224)
b6b061b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@end-4 @clsty