Closed mazzystr closed 2 years ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: To complete the pull request process, please ask for approval from mazzystr after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.
We can't. All the icons are governed by corporate legal/marketing. They're giving me the exact size in px on how they want the logo displayed.
On Tue, Sep 7, 2021 at 7:38 PM Tyler Auerbeck @.***> wrote:
@.**** commented on this pull request.
LGTM. Only thing I'd mention (and maybe something to look into in the future) would be normalizing the icon sizes. Right now they look a bit all over the place.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/kubevirt/kubevirt.github.io/pull/794#pullrequestreview-748576001, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAS5KZZORB6MZG6S6A6RD2TUA3EEDANCNFSM5DS7QCDQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
Preview link is here .... https://deploy-preview-794--kubevirtio.netlify.app/
@mazzystr: PR needs rebase.
Signed-off-by: Chris Callegari mazzystr@gmail.com
Fixes #782
Special notes for your reviewer: @mlsorensen