-
Notifications
You must be signed in to change notification settings - Fork 208
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
Rebrand /landscape #14540
Rebrand /landscape #14540
Conversation
ty @immortalcodes , only a couple things from me!
For this section (smaller screens):
|
Thanks @immortalcodes! One question on the 'Get Landscape' CTAs throughout the page - it links to the /pricing page, which will be changed to /compare instead. Is this something we'd need to change on the index page now? Also:
|
|
thanks @immortalcodes ! literally just two v small change to be made, if possible, so I'll +1: this section is missing a regular bottom padding: could the break after "scale" on smaller screens be removed pls, as it ends up with the sentence occupying 3 lines: |
Noted on the links, will keep an eye out when the page goes into development. You're right - the CTA block should remain as H2s so please leave as is. One more comment - would you mind adding 'Explore IoT device management' as a primary CTA under 'Discover Landscape |
Thanks for the changes @immortalcodes! Adding +1 |
Done
QA
./run serve
ordotrun
Issue / Card
Fixes # WD-16834
Help
QA steps - Commit guidelines