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

Toggle button for A/B testing #21

Merged
merged 17 commits into from
Mar 27, 2023
Merged

Toggle button for A/B testing #21

merged 17 commits into from
Mar 27, 2023

Conversation

oraNod
Copy link
Contributor

@oraNod oraNod commented Mar 3, 2023

Fixes #20

@oraNod oraNod requested a review from samccann March 3, 2023 12:03
@oraNod
Copy link
Contributor Author

oraNod commented Mar 3, 2023

@samccann It's still a bit rough but what do you think about this approach? We can add all the html from the current docsite to an olddocsite subdirectory and add a toggle button with a bit of js.

@oraNod oraNod changed the title Toggle button for A/B testing [WIP] Toggle button for A/B testing Mar 3, 2023
@samccann
Copy link
Contributor

samccann commented Mar 3, 2023

I'd change 'toggle to old site' to 'return to old site'. And I clarified in the issue but I'll repeat here - toggle goes to/from the actual live docs.ansible.com. That way we get live people (when the time comes) seeing a much larger call to action button on the top of that page. Something like "Try our new docsite redesign!" so it's big and obvious. That brings hopefully hundreds of visitors to the new docsite, where we will need a feedback button or something so they can tell us what they thing and wht needs fixing so to speak.

@oraNod oraNod changed the title [WIP] Toggle button for A/B testing Toggle button for A/B testing Mar 22, 2023
@oraNod
Copy link
Contributor Author

oraNod commented Mar 22, 2023

@samccann Could you please take another look? Not sure how great this is tbh.

oldsite/static/js/toggle.js Outdated Show resolved Hide resolved
oldsite/static/js/toggle.js Outdated Show resolved Hide resolved
oldsite/static/js/toggle.js Outdated Show resolved Hide resolved
oldsite/static/js/toggle.js Outdated Show resolved Hide resolved
static/js/toggle.js Outdated Show resolved Hide resolved
@oraNod
Copy link
Contributor Author

oraNod commented Mar 23, 2023

@felixfontein Great suggestion with the data field to set a redirect page name. That works like a charm and is certainly better than copying pages during the build.

@samccann
Copy link
Contributor

This is looking pretty good! Can we make the 'try out the new site' text either bigger or in bold? We want anyone on the 'old site' pages to really know there's something better out there lol.

As a side question - we publish all these pages to docs.ansible.com and the new pages are the default with this, and the old pages accessible by that toggle?

@oraNod
Copy link
Contributor Author

oraNod commented Mar 27, 2023

@felixfontein Thanks for the great suggestions and the help simplifying the page toggle. I can honestly say I've learned from that and it's been fun.

@oraNod
Copy link
Contributor Author

oraNod commented Mar 27, 2023

@samccann I've updated the wording and style for the toggle. Let me know if you think it needs anything else. Thanks for the great reviews as always.

Copy link
Contributor

@samccann samccann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

output lgtm thanks

@samccann samccann merged commit 89a1acf into ansible:devel Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create ability to A/B test our docsite
3 participants