feat(Image): add hide param #1182
Conversation
|
Playwright Test Component is ready. |
|
Preview is ready. |
aeksandla
force-pushed
the
aeksandla/hide-image
branch
from
e576511 to
258a018
Compare
| return ( | ||
| <picture className={containerClassName} data-qa={qa}> | ||
| {mobile && ( | ||
| {(mobile || hideDevices.mobile) && ( |
There was a problem hiding this comment.
if hideDevices.mobile === true I think we don't need render this block, but It's the opposite now
maybe we need to change name for hideDevices
There was a problem hiding this comment.
We show this block when the picture in mobile is hidden to show an empty picture. If you do not show this block when the hideDevices.mobile === true parameter, a desktop or tablet image will be installed on the mobile resolution.
| qa={qaAttributes.tabletSource} | ||
| /> | ||
| )} | ||
| {hideDevices.desktop && ( |
There was a problem hiding this comment.
where case for hideDevices is boolean true ?
There was a problem hiding this comment.
In line 135 it says that an empty picture is displayed here. If the image on the desktop is hidden, then we show an empty picture.
No description provided.