-
Notifications
You must be signed in to change notification settings - Fork 66
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-13772 Refresh /data/postgresql #1362
WD-13772 Refresh /data/postgresql #1362
Conversation
Logo section here needs a couple of changes. The google cloud logo is deprecated, so pls use this one instead. Also, MAAS logo is different size from the MicroCloud logo, so just be sure to use this one, taken from the "logo-library-07-24-updated" server For images with text underneath, pls use 24px padding like in the fiigma, just to add a bit of breathing room there. As you mentioned on MM, there's still an issue with the image under "PostgreSQL support", but I'll look into it. And pls remove the h5 "support" there. For this section, the latter three components have a bg that's too dark and there's a gap(?) between the bold line and the container, but I suppose this isn't an issue if bg colour is the same. Also, link under first comp should be entirely blue. |
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.
Left some small change suggestions
If I remove the highlight class, this is what I get. The latter images have a background of their own and hence we see difference in colors and a space between the line The text for the first image has 2 different links, I couldn't have one of them over whole text. |
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.
Great work, thanks for making the changes! 🎉
Done
QA
./run serve
Issue / Card
Fixes # WD-13772