[MPT-57] Home: Standardise vertical alignment of content #703
Conversation
✅ Deploy Preview for tender-meitner-99286b ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
wei2912
requested review from
a team and
nicholasohjj
and removed request for
a team
There was a problem hiding this comment.
Vertical alignment fixed.
Mentor tab needs padding and margin from top of the card. Reduce border radius.
Avatar image should be shifted to the left to align with testimonial titles.
More padding between avatar image and right text content.
Consider align left for name + tag, an
d title to align with quote.
|
To be routed to @jcc-kh for final verdict on design, after @nicholasohjj clarifies feedback above with @Unknownflow. |
There was a problem hiding this comment.
@Unknownflow See below comment from @jcc-kh:
Is it possible to have the words 'Managing Director' and 'Persistensie Pte Ltd' on the same line? Right now it's too sparse it looks a bit weird
|
@Unknownflow can attach the updated screenshot of the page here for review |
|
|
|
Think having them directly side-by-side would impact readability, @jcc-kh do you have suggestions for how they can be arranged on the same line (keeping in mind that the design should work on mobile too)? |
|
HM, add a comma in between the words? HAHA I think that's how people do it? |
|
I have added a comma between the words, in this case it is between the mentor role and company as shown below.
|
|
Sorry, think I neglected to get this merged in before working on 3cd8463... will take over the PR. |
|
@Hackin7 Handing over to you in case any of the code here is relevant to the latest version. |
I aligned the headers of the content so they are in line with each other