-
Notifications
You must be signed in to change notification settings - Fork 108
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
Move model overview CTA and update title #4373
Conversation
Demo starting at https://snapcraft-io-4373.demos.haus |
Codecov Report
@@ Coverage Diff @@
## main #4373 +/- ##
=======================================
Coverage 80.32% 80.33%
=======================================
Files 205 206 +1
Lines 10273 10276 +3
Branches 901 901
=======================================
+ Hits 8252 8255 +3
Misses 1899 1899
Partials 122 122
Flags with carried forward coverage won't be shown. Click here to find out more.
|
08fba93
to
a6cae89
Compare
@steverydz I agree with Luke on the breadcrumb (however, there is no dot in my preview). Secondly, could you please move the first text line up to align with the text on CTAs? Thanks in advance! |
a6cae89
to
f809947
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.
👍 LGTM
Everything looks good, thanks Steve |
Done
How to QA
Issue / Card
Fixes https://warthogs.atlassian.net/browse/WD-5631