WD-16838 Rebranded /landscape/managed page

[muhammad-ali-pk]

Done

• Rebranded page

QA

• View the site in your web browser at: https://ubuntu-com-14580.demos.haus/landscape/managed
  • Be sure to test on mobile, tablet and desktop screen sizes
• Check that it matches the following:
  • Design
  • CopyDoc

Issue / Card

Fixes #WD-16838

[webteam-app]

Demo

Jenkins

demos.haus

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (feature-rebrand-landscape@32026ad). Learn more about missing BASE report.

Additional details and impacted files

@@                     Coverage Diff                      @@
##             feature-rebrand-landscape   #14580   +/-   ##
============================================================
  Coverage                             ?   69.64%           
============================================================
  Files                                ?      120           
  Lines                                ?     3419           
  Branches                             ?     1174           
============================================================
  Hits                                 ?     2381           
  Misses                               ?     1013           
  Partials                             ?       25           

[mattea-turic]

Thanks @muhammad-ali-pk ! Only a couple of things from me:

• For the section "Bring your own identity and access management platform", the image should have the grey background – I forgot to apply that back on figma after exporting the image, so mb!
• I'd also say to hide that image on smaller screens as it's there's too much detail for it to work unless you're viewing on a desktop (or large tablet)
• You forgot this section: "Mirror Ubuntu repositories, and distribute software internally"

[muhammad-ali-pk]

Thanks @mattea-turic! All done!
Though strange, that section feeling like déjà vu.

[eliman11]

Thanks @muhammad-ali-pk, this looks great! Just a few quick comments:

• Link asterisk(*) in the pricing block to the live Managed Apps page instead of the copydoc for that page, also added as a suggestion in the copydoc - https://ubuntu.com/managed/apps
• Change the [Get started with a free quote and architecture assessment ›] and the [Contact us about your unique requirements ›] CTAs to link to the contact form instead of the copydoc
• Could you also add alt text to logos in the logo cloud using the titles on the copydoc?

[muhammad-ali-pk]

@eliman11 Thanks Elaine. All done!

[Sophie-32]

1. All images are decorative, should have alt=""
2. Added a small item in suggested mode in the copydoc and tagged @muhammad-ali-pk in the Design as well

[muhammad-ali-pk]

@Sophie-32 Thanks Sophie! I have added the suggestion.
Just a tiny confusion on the images. All the images except the brand logos for cloud are decorative and have alt="". However, shouldn't the logos actually have the alt text, especially since brand logos usually don't count as decorative, and also, someone using Text to Speech would be needing those alt texts for the logos?

[Sophie-32]

@muhammad-ali-pk - thanks for flagging. Yes for logos our guidance is to "se the brand name (for example, ‘Microsoft Azure,’ not ‘Microsoft Azure logo’). Use empty state if the company name is used in the title or a link label."

[mattea-turic]

Hey @muhammad-ali-pk , just noticed you've got an extra asterix undet the table in "Pricing that is clear and transparent"

I'll +1 for design, though!

[immortalcodes]

We can remove the p-section and p-section--shallow here and just add p-section

[immortalcodes]

Shallow padding here

[immortalcodes]

Thanks for the PR. Good Work!
Left minor comments

[immortalcodes]

Now we have a shallow padding here so we don't need u-sv4

[immortalcodes]

LGTM! Please do that one change and it should be good to merge