Bonjourr icon indicating copy to clipboard operation
Bonjourr copied to clipboard

Save button for the unsplash wallpaper

Open baozidai opened this issue 2 years ago • 11 comments

Is your feature request related to a problem? Please describe. When I get rolled a nice image from unsplash, I want to save it, but I can't found it :( Now I can only get to the site of the photogragher, but can't find the image.

Describe the solution you'd like Add a button for save current unsplash image to local computer. It can be a low priority feature maybe.

baozidai avatar Mar 04 '24 18:03 baozidai

just tap the name of the wallpaper in the bottom left corner it will redirect you to the unsplash website and you can download it from there

rajarshikhatua avatar Mar 05 '24 03:03 rajarshikhatua

Don't close this issue. I found a localisation problem. In zh_CN, there is no "photo"-like word to indicate that you can access the photo. Left corner shows "拍摄者 " which means "photographer is " but in en_us shows "Photo by " which "Photo" notice you can click it. I hope you can understand the above misunderstanding. I will fix the translation soon.

baozidai avatar Mar 05 '24 05:03 baozidai

What I want it's "图片 由 !someone! 拍摄". In this way it just like "Photo by !someone!" in English which "Photo" mapping to "图片" then it click-able. But I can not make it by only editing the localization because of the tail "拍摄".

Honestly,it is OK there is not "拍摄", but it will be strange in Chinese.

baozidai avatar Mar 07 '24 18:03 baozidai

I see. There is a similar issue with the weather forecast in Russian, which means we'll have to improve the way full sentences are translated.

It shouldn't be too complex, however I've started a lot of other things I need to finish first, so that might take some time.

I will @ you in this issue once the new translation method is available !

victrme avatar Mar 07 '24 18:03 victrme

Hello @baozidai, improved credit translation is here! Is this translation correct ?

image

victrme avatar Mar 21 '24 07:03 victrme

@victrme Absolutely right! Thanks for your efforts!

baozidai avatar Mar 21 '24 08:03 baozidai

I keep coming back to this feature request, and I just wanted to mention I think it has potential.

@rajarshikhatua100 is correct: one can click on the link and download the background image from the source. But that's an extra step (and more MBs for people on slow/metered connections).

My purpose of posting is just to mention that I think a "save" button for the background may be worthwhile. Another possibility could be to add right-click functionality (on the link that opens the background's source page) that quickly downloads the image.

In any case, I don't think it's a big priority, but this keeps coming up in my feed because I keep thinking about it and wanted to mention my thoughts... and now I have. 😄

Gitoffthelawn avatar May 03 '24 07:05 Gitoffthelawn

I agree it does save a few clicks and page loads! I think we can add a small download icon that appears when you hover your mouse over the credits.

We also have some ideas regarding right clicks, so that might also be added in the future!

victrme avatar May 03 '24 13:05 victrme

@victrme would something like that do the job?

ChainsLunatic avatar Jun 16 '24 19:06 ChainsLunatic

Hi @ChainsLunatic, there are a few things to change, but this is the expected behavior !

First, I was thinking more of an icon rather than text. That way we don't need to translate it and it takes less space.

Second is how we trigger a download. We have to follow the guidelines for triggering a download, which means instead of using the slug, we need to use the download_location field. You can access the endpoint using apiFetch:

const download = await apiFetch("/unsplash/photos/<photo_id>/download?ixid=")

victrme avatar Jun 19 '24 06:06 victrme

Allright, thanks for your feedback! I'll look into it :)

ChainsLunatic avatar Jun 19 '24 07:06 ChainsLunatic