-
Notifications
You must be signed in to change notification settings - Fork 101
KDropdown and KIconButton tweaks #465
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
base: develop
Are you sure you want to change the base?
Changes from all commits
f44d921
5c05d55
abb8af1
378ab40
f3c7e42
c016d7d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,6 +7,17 @@ Changelog is rather internal in nature. See release notes for the public overvie | |
|
||
## Version 2.0.0 | ||
|
||
- [#465] | ||
- **Description:** KDropdownMenu improvements & bind $attrs in KIconButton | ||
- **Products impact:** - | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this change would fall into "updated API" products impact category since there are some updates to public facing API for components like a new slot and attributes binding |
||
- **Addresses:** - | ||
- **Components:** KDropdownMenu, KIconButton | ||
- **Breaking:** no | ||
- **Impacts a11y:** yes | ||
- **Guidance:** The KIconButton simply allows you to ensure that attrs you bind to it will be applied to the resulting button itself (ie, tabindex). KDropdownMenu now exposes a #option slot. This slot exposes the current option so a template snippet can be passed here to customize the display of items in the menu. This could also be used to show a list of checkboxed items or perhaps a KCircularLoader next to an item. | ||
|
||
[#465]: https://github.com/learningequality/kolibri-design-system/pull/465 | ||
|
||
- [#478] | ||
- **Description:** Changed _dev-only to dev-only | ||
- **Products impact:** - | ||
|
@@ -15,7 +26,7 @@ Changelog is rather internal in nature. See release notes for the public overvie | |
- **Breaking:** no | ||
- **Impacts a11y:** no | ||
- **Guidance:** - | ||
|
||
[#478]: https://github.com/learningequality/kolibri-design-system/pull/478 | ||
|
||
- [#482] | ||
|
@@ -26,7 +37,7 @@ Changelog is rather internal in nature. See release notes for the public overvie | |
- **Breaking:** no | ||
- **Impacts a11y:** no | ||
- **Guidance:** - | ||
|
||
[#482]: https://github.com/learningequality/kolibri-design-system/pull/482 | ||
|
||
- [#464] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,10 @@ | |
Implements a dropdown set of options, based on <DocsExternalLink text="Keen's UI Menu" href="https://josephuspaye.github.io/Keen-UI/#/ui-menu" />. See these docs to understand the current implementation of the options object array. | ||
</p> | ||
<p> | ||
Notable possible for configuring the menu include: icons, text, secondary text, and dividers. | ||
Notable possible for configuring the menu include: icons, text, secondary text, and dividers. Additionally, you can customize the appearance and render complex options using the "options" slot and its associated slot-props. | ||
</p> | ||
<p> | ||
The dropdown will typically contain the user's focus while they use the arrow keys to navigate the options. The <code>Tab</code> and <code>Esc</code> keys will close the menu and emit a <code>@close</code> event. The <code>Tab</code> key will pass along the browser event that triggered the close will be emitted by way of a <code>@tab</code> event on the <code>KDropdownMenu</code> which can be useful for managing focus after the menu is closed. | ||
Comment on lines
+10
to
+13
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @radinamatic curious if this is true - could be basis for future issue to update our behavior and our documentation if there are discrpencies |
||
</p> | ||
<p> | ||
Please see the <DocsInternalLink href="/buttons#dropdowns" text="Dropdown section of the Buttons and links page" /> on the buttons page for more details about how to use with a button, and a code example. | ||
|
Uh oh!
There was an error while loading. Please reload this page.