refactor(knobs): Gallery optional/mandatory knobs -FRONT-926 #392
Conversation
data.overlay.share.icon.path = defaultSprite; | ||
const preperaGallery = data => { | ||
data.items.forEach((item, index) => { | ||
item.share_path = select( |
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.
What about the other parameters?
- "path"
- "alt"
- "meta"
- "icon"
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.
i didn't see them in the documentation in ecl for the gallery but i will put them in
defaultSprite, | ||
buttonLabels.required | ||
); | ||
data.overlay.download.icon.path = select( |
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.
this is a link object, not only the icon.path should be a knob
defaultSprite, | ||
buttonLabels.required | ||
); | ||
data.overlay.share.icon.path = select( |
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.
this is a link object, not only the icon.path should be a knob
PR description
Please drop a few lines about the PR: what it does, how to test it, etc.
QA Checklist
In order to ensure a safe and quick review, please check that your PR follow those guidelines:
devDependencies
devDependencies
include
orembed
) asdependencies
@ecl-twig/ec-components
'sdependencies
yarn.lock
contains changes, I have committed itpr: review needed
to indicate that I'm done and now waiting for a review ,pr: wip
to indicate that I'm actively working on it ...)