Style claim and invest page. #2294
Conversation
|
src/custom/pages/Claim/styled.ts
Outdated
| } | ||
|
|
||
| > span::before { | ||
| content: 'Step ' attr(data-step); |
There was a problem hiding this comment.
i don't personally like this it makes it quite hard to change later on if we forget it's in the CSS. Can't you just write this out in the HTML of InvestmentFlow?
There was a problem hiding this comment.
@W3stside Agreed, addressed now.
|
Looks great besides the stepper in a mobile view. |
|
point this to |
| </StepExplainer> | ||
| <p> | ||
| For more details around the token, please read{' '} | ||
| <ExternalLink href={COW_LINKS.vCowPost}>the blog post</ExternalLink>. For more details about the claiming |
There was a problem hiding this comment.
nit: should we give a line break for the other For more details... ?
There was a problem hiding this comment.
Yes or we could try to write a single sentence pointing to both links (don't have a suggestion for that now :) )
|
@elena-zh Pushed a fix for that: |
Will review in a separate PR. Might have addressed it already. |
|
@biocom , the 2nd step text is left-aligned now on all types of mobile devices when screen width is less than 395 px |
|
@elena-zh should be fixed now. Merging to expedite, feel free to address post merge. |
|
Looks good, thanks! |
Summary