Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Add top bar for former CC Search users #194

Closed
zackkrida opened this issue Sep 7, 2021 · 6 comments · Fixed by #212
Closed

Add top bar for former CC Search users #194

zackkrida opened this issue Sep 7, 2021 · 6 comments · Fixed by #212
Assignees
Labels
💻 aspect: code Concerns the software code in the repository 🌟 goal: addition Addition of new feature good first issue New-contributor friendly help wanted Open to participation from the community 🟧 priority: high Stalls work on the project or its dependents 🏁 status: ready for work Ready for work

Comments

@zackkrida
Copy link
Member

Anyone visiting wordpress.org/openverse from a domain containing creativecommons.org should see a special banner at the top of the site. This is intended so that CC Search users aren't confused when they are redirected to Openverse, and so we can control the narrative about what a positive change for the project this is.

Mockup

CleanShot 2021-09-06 at 21 26 26@2x

Figma link: https://www.figma.com/file/BlUTIXRHLOq5mRITpDpi3E/Transitional-styles?node-id=418%3A2025

Please only show if the referrer is from a creativecommons.org domain. And make sure to localize the text. Link should go to https://wordpress.org/news/2021/05/welcome-to-openverse/

@zackkrida zackkrida added good first issue New-contributor friendly help wanted Open to participation from the community 🟧 priority: high Stalls work on the project or its dependents 🏁 status: ready for work Ready for work 🌟 goal: addition Addition of new feature 💻 aspect: code Concerns the software code in the repository labels Sep 7, 2021
@dhruvkb dhruvkb self-assigned this Sep 14, 2021
@dhruvkb
Copy link
Member

dhruvkb commented Sep 14, 2021

@panchovm I have a few questions about this bar.

  • Should this bar be dismissable, for example, with an X icon?
  • Will this be shown again to the same user if they follow the redirect again?

@dhruvkb
Copy link
Member

dhruvkb commented Sep 14, 2021

@zackkrida what approach will CC be employing to perform the redirect? It's very likely that the referrer will be empty unless the user reaches the page by clicking a link. All approaches like 301/302 redirects, JavaScript window.location are leading to the referrer being blank.

@zackkrida
Copy link
Member Author

Typically a 301 preserves the referer, but I suppose we could also have them add a query parameter or other more reliable method of showing the banner!

@dhruvkb
Copy link
Member

dhruvkb commented Sep 14, 2021

Even if 301 would preserve the referrer, wouldn't that contain the domain that sent them to CC Search then, and not the CC Search domain itself? Query param would be awesome 👍 if they can do that. Our theme anyways has the path forwarding set up so it'll reach the Vue iframe automatically.

@zackkrida
Copy link
Member Author

OH duh. This whole time I've been conflating the original referrer to search.creativecommons.org with search.creativecommons.org itself. We will definitely have them add a query param, thank you!

@fcoveram
Copy link

Arriving late here, but coming to say that we talked about having the message for a period of time (TBD).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code Concerns the software code in the repository 🌟 goal: addition Addition of new feature good first issue New-contributor friendly help wanted Open to participation from the community 🟧 priority: high Stalls work on the project or its dependents 🏁 status: ready for work Ready for work
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants