-
Notifications
You must be signed in to change notification settings - Fork 12
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
FRONT-2475: Dropdown pattern. #50
Conversation
eca8f07
to
90c9424
Compare
623e934
to
4e33c6e
Compare
313ef2a
to
8583410
Compare
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.
Small thing (see below) and rebase pls.
also look into templates/overrides/navigation/links--dropbutton.html.twig, maybe we need to adapt ? I'm not imposing though, up to you.
{# | ||
/** | ||
* @file | ||
* Dropdown component. |
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.
* Dropdown component. | |
* Dropdown pattern. |
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 tried to adapt but I think is not possible. Main reasons:
- The button for instance sometimes is outlined but at dropdown we cannot set this variant,
- Links are rendered link objects, so is not possible to pass as variable
…check for the items url in patterns.inc.
…t dropdown.ui_patterns.yml.
8583410
to
c92d87a
Compare
No description provided.