176 demo account #159
176 demo account #159
Conversation
| {paragraphs.map((par, index) => ( | ||
| <div className={`text-box--${alignment} ${addClass || ''}`} key={index}> | ||
| <p | ||
| className={`text-box text-box--${size} mb-0`} |
There was a problem hiding this comment.
when we have multiple paragraphs, we want space between them. I initially had a conditional to remove the spacing from the last paragraph only, but felt that was too specific to our current use case. (part of the "about us" section).
in the future, this component could be used anywhere, and having the bottom margin is a good thing.
| <> | ||
| <Title title='My Requests' size='medium' /> | ||
| <div className='rounded overflow-hidden'> | ||
| <div className='rounded overflow-hidden mb-4'> |
There was a problem hiding this comment.
adds some spacing between the last item in the list, and the footer.
…ersions of the secondary color. plus formatting in this file
…info" form headers, dynamic
| </div> | ||
| ) | ||
| })} | ||
| const StatusBar = ({ backgroundColor, statusArray, apiRequestStatus, addClass }) => { |
| import './status-bar.scss' | ||
|
|
||
| const StatusBar = ({ statusArray, apiRequestStatus, addClass }) => ( | ||
| <div className={`container ${addClass}`}> |
There was a problem hiding this comment.
the majority of this code is the same. I needed to create a new variable and function though, so the return body is no longer immediately being returned. a lot of this change is adding indention for that reason.
| return ( | ||
| <div | ||
| className={`status-bar-column col-3 py-2 gap-2 d-flex justify-content-center align-items-center | ||
| ${border} ${labelBgColor(index)}`} |
There was a problem hiding this comment.
deriving both of these classes from the new code above.
ref: scientist-hq/webstore#176
expected behavior