-
Notifications
You must be signed in to change notification settings - Fork 137
Conversation
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.
Looks ok but did you test with varying string length for translated strings? Perhaps you could provide a link via running now
if you're setup with that and we can test.
@ryanml there is new design for it, can you please update it? brave/brave-browser#1910 (comment) |
255d09a
to
4225c44
Compare
@ryanml please use flex instead of inline-block, so that this scenario is handled as well |
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 feedback
export const StyledMonthlyTips = styled<{}, 'span'>('span')` | ||
color: #696FDC; | ||
font-weight: 400; | ||
display: inline-block; |
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.
we need this one
Fixes: #252
Related: brave/brave-browser#1910
Edit, updated design: