ipfs / ipfs-desktop

An unobtrusive and user-friendly desktop application for IPFS on Windows, Mac and Linux.
https://docs.ipfs.tech/install/ipfs-desktop/
MIT License
5.96k stars 861 forks source link

Improve UX for "auto add screenshot" #765

Open momack2 opened 5 years ago

momack2 commented 5 years ago

I love the auto-add screenshot function - but I think it could use some UX smoothing:

I bet these are probably shared desktop/webUI items - but found them in this context so figured I'd surface them here. =] Great work! Looking forward to becoming a regular user of this feature

hacdias commented 5 years ago

Hey @momack2! Glad you liked it and brought some feedback. I'm splitting this issue into two (this one here and https://github.com/ipfs-shipyard/ipfs-webui/issues/917 for Web UI's issues).

When taking a screenshot, send a notification that users can click to see their screenshot in the desktop UI

Yup, that can be done and I that was what was initially planned when this featured was introduced (like one year ago). But at the time there was a bug that didn't send notifications on Windows, but that is solved now.

Have a step between keyboard shortcut and selecting what to screenshot (full desktop, active window, choose selection) - otherwise I might accidentally screenshot and start providing something confidential!

I know we an screenshot the whole screen and Electron's window. I wonder if we can get a good screenshot of windows of other apps. I will investigate.

I get a 524 (or worse, an infinite loading screen) when trying to "share" and view a screenshot I just took on the ipfs or cloudflare gateways

I believe this is #750.


We'll work into these. Thank you 😄 Any other ideas?

/cc @ipfs-shipyard/gui