-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Add a hero
component
#99
Conversation
Export SizeUnchanged@studiometa/ui
|
Codecov Report
@@ Coverage Diff @@
## develop #99 +/- ##
===========================================
- Coverage 26.64% 26.55% -0.10%
===========================================
Files 46 46
Lines 1141 1145 +4
Branches 211 214 +3
===========================================
Hits 304 304
- Misses 771 775 +4
Partials 66 66
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Nice first draft !
suggestion Add a block to insert breadcrumbs |
…tes instead of just classes. Add conditions to the elements. Add the figure componant. Add the possibility to have mutliple CTA
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.
Hey @Liax, great job on the update!
Note: don't be overwhelmed by the multiple comments I left, this component is looking really good and promising 👍
This should simplify getting started with the project.
{ class: ['hero__image'] } | ||
) | ||
}) | ||
%} |
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 some additionnal params, see below :
{% set image =
image
|default({})
|merge({
src: image.src,
width: image.width,
height: image.height,
alt: image.alt,
attr: merge_html_attributes(
image.attr ?? null,
{ class: 'absolute inset-0' },
{ class: 'hero__image' }
)
})
%}
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.
One small adjustment to the CHANGELOG and we will be able to merge this! Well done @Liax!
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.
👏
Objective
First draft of a hero component
To do:
...