-
Notifications
You must be signed in to change notification settings - Fork 20
Attribution & Giphy Guidelines #36
Comments
Hi again @antoinejaussoin and thanks for reporting 🙂 . At the time I developed this module they haven't released this guidelines yet, but I think we can add what they're asking. Do you think it could be enough to add the name of the contributor for each result (linked with the original media on Ghiphy's website), or do you believe we should implement a more complex solution, like the one they are showing in their docs? I understand the needing of give the right attribution to the authors, but I don't want to add too much complexity to the UI, keeping it simple and focused in with the main purpose. |
@antoinejaussoin I quickly drafted a possible solution (take a look at the first two results in the below image), but I don't like it so much. Adding the name over the image, in our case, significantly covers the image itself. Otherwise, showing it on mouse over, forces us to manage differences between non-touch and touch devices. Any suggestions or feedback? |
A further thought: opening their website at the trending page, I can't see any attribution to authors showed over the images on mobile/touch view, neither on the Anyway... following what they are doing on their website, probably a compromise solution could be to hide the attribution on mobile/touch view and show it on the |
Hi! Apologies for the late reply. I was wondering if a simple tooltip over the image wouldn't be enough? As you said, it would only work on desktop but it's probably fine? |
@antoinejaussoin I understand the purpose of giving the proper attribution to authors asked by Giphy's team, but this module is intended to have a quick search and select tool, I don't want the user can be distracted by unuseful information. I think that the idea I proposed could fit the main need (as well the tooltip, it’s just an aesthetic issue). I can add it, maybe leaving it optional with a prop, but we have to figure out if the Giphy's dev team will like the way we will add this feature. |
I could be wrong but I think you just need a updated logo? https://developers.giphy.com/faq/ What conditions does my app/project need to meet in order to get a production API Key?
https://support.giphy.com/hc/en-us/articles/360028134111-GIPHY-API-Terms-of-Service- Approved logos: https://media.giphy.com/giphy-attribution-marks.zip |
I don't think so @caleb15, I'm using the |
Hi!
Thanks for your amazing component, which I'm using on retrospected.com.
I've just received an email from Giphy, stating that the component doesn't comply to the Attribution Policy:
Looking at their website (https://developers.giphy.com/docs/sdk/#design-guidelines), it looks like the component needs to somehow display the image creator's handle/username and perhaps a link to see more from them:
My question is: is that something you are planning to work on at some point?
Thanks!
The text was updated successfully, but these errors were encountered: