-
Notifications
You must be signed in to change notification settings - Fork 194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve landing page #1023
Improve landing page #1023
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some quick observations / change requests:
- "Powered by" should be centred along with the logo
- MMM table. Great, but can we make it "Feature" and "Benefit" instead of description.
- I like the inclusion of some badges
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really great. Quick Look
Thanks @drbenvincent ! I addressed all of your comments 🙏 (we can do something similar for CausalPy if you want) |
That would be great 👍🏻 |
441faa4
to
dac633e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still think the landing page has a huge amount of information on it (e.g., the page is very long), but at least there is a bit more Labs content and it is nearer to the top of the page now.
@cluhmann Agree. Do you think some of the videos can be removed? Which ones? |
Co-authored-by: Christian Luhmann <[email protected]>
Maybe the "Schedule a Free Consultation for MMM & CLV Strategy" and "Support" sections could be trimmed (I can make concrete suggestions). The "Examples" under the CLV section (the 2x2 table) doesn't seem to do anything? Is this supposed to include links? Then, under "Resources" I think we can maybe have 1 video and then a link to a page with the rest of the videos? Or to the playlist on YouTube? |
@cluhmann I actually removed the Support section as it seemed redundant. I left just three videos: a business case, an mmm and a clv one, WDYT? Besides that, I think I have addressed all your comments. |
Let me just do some editing on the "Schedule a Free Consultation" section. Should get to it in a few hours. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's see how these changes look. Not as compact as I had hoped, but a bit better.
Co-authored-by: Christian Luhmann <[email protected]>
Co-authored-by: Christian Luhmann <[email protected]>
Co-authored-by: Christian Luhmann <[email protected]>
Co-authored-by: Christian Luhmann <[email protected]>
Co-authored-by: Christian Luhmann <[email protected]>
Thanks @cluhmann ! I think is ready for a last review :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just the 1 fix I see.
Make the landing page less dense and add a get-started section
📚 Documentation preview 📚: https://pymc-marketing--1023.org.readthedocs.build/en/1023/