Conversation
|
@siriwatknp Oops, I was initially aiming to remove the duplication (x3 |
Okay, no prob. I have brought it back. I don't think there is anything left for this page (checked the lighthouse in the PR description, it looks good to me) |
Font loaded: In https://next.material-ui.com/branding/about/, we fixed it by forcing the line break where we want it (with a
For instance, should it have the ripple when focused or an outlined as in https://next.material-ui.com/branding/about/
|
|
@oliviertassinari For 4 and 5, I am waiting to hear from @danilo-leal opinion but I don't think it is a priority and can be improved later. |
oliviertassinari
left a comment
There was a problem hiding this comment.
Awesome, I'm holding on the reviews so we can get a first iteration in :)
About 4, what would you make it different? And about 5, we're going to revisit that in a refinement phase (updating the companies we use + the coloring and sizing of the logo - to all pages). Agree with not being a priority :) Very awesome work! |
…branding/about
|
I'm consolidating a list of feedback that we could handle for the polish phase:
|
|
I recall @mbrookes did it for the current logos on the homepage, it felt better. |
Decorating the logo also goes against the visual identity guidelines (ours, and in general). @danilo-leal let me know if you don't have them. |










Preview: https://deploy-preview-27599--material-ui.netlify.app/branding/about/
waiting for #27598
added tooltip to show location on hover. I think it is nice to see the location apart from country flag.
Lighthouse on desktop
