Skip to content
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

WD-17690 Refresh https://ubuntu.com/azure/pro #14636

Merged
merged 5 commits into from
Jan 10, 2025
Merged

WD-17690 Refresh https://ubuntu.com/azure/pro #14636

merged 5 commits into from
Jan 10, 2025

Conversation

bartaz
Copy link
Member

@bartaz bartaz commented Jan 10, 2025

Done

Refreshed /azure/pro page with new content updates and added latest blog section.

Copy doc: https://docs.google.com/document/d/1WaQ5n3GxwZzbX0T6rLiRNejFvxIPIHd5V5XgdAwKmSw/edit?tab=t.0
Figma: https://www.figma.com/design/MinLlXouBnEDqaoAL3bTWC/24.10-ubuntu.com%2Fazure?node-id=1233-2555&t=QY6LrkH2N8fT497L-0

QA

Issue / Card

Fixes WD-17690

Screenshots

image

Help

QA steps - Commit guidelines

@webteam-app
Copy link

@eliman11
Copy link

eliman11 commented Jan 10, 2025

Thanks @bartaz! A couple comments:

  • Change to US spelling: optimised -> optimized in "Predictable and optimised"
  • Would we be able to add the chevron on top of the blog section to scroll through all the articles with the "Ubuntu on Azure" tag?
Screenshot 2025-01-10 at 11 25 55

@bartaz
Copy link
Member Author

bartaz commented Jan 10, 2025

  • Would we be able to add the chevron on top of the blog section to scroll through all the articles with the "Ubuntu on Azure" tag?

@eliman11 Do you know if we have such functionality anywhere else already? Adding this as a new feature seems a bit outside of a BAU refresh scope.

@bartaz
Copy link
Member Author

bartaz commented Jan 10, 2025

As discussed in Mattermost chat, it seems that the carousel functionality is not implemented yet and is being skipped in this PR.

@eliman11
Copy link

Thank you for the quick changes! Adding UX+1

anthonydillon
anthonydillon previously approved these changes Jan 10, 2025
Copy link
Contributor

@anthonydillon anthonydillon left a comment

Choose a reason for hiding this comment

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

Code and QA 👍
Wave has no complains about the content.

@mattea-turic
Copy link
Collaborator

mattea-turic commented Jan 10, 2025

Thanks @bartaz ! Looks great, just a couple of comments:

For "Upgrade Ubuntu LTS to Ubuntu Pro"

  • Could you add a break after "LTS" pls?

For "Is Ubuntu Pro right for you?"

  • There looks to be too little spacing between this section and the one below. Would it be possible to use the same wrapping or the link here as you have for the sections above – as together with the shallow, it'll add some breathing room

Either way, these are v small so I'll +1 for design

Copy link
Contributor

@anthonydillon anthonydillon left a comment

Choose a reason for hiding this comment

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

Still good. Little abuse of the   but tested and not harmful until very small screens.

@bartaz bartaz merged commit 2da81ec into main Jan 10, 2025
29 checks passed
@bartaz bartaz deleted the WD-17690-azure-pro branch January 10, 2025 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants