-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add access key to button #5482
Add access key to button #5482
Conversation
…ure-shotcuts-buchstabe-hinter-label-anzeigen * 'develop' of github.com:public-ui/kolibri: restore: toast readme for documentation v2.0.0-rc.4 Update snapshots Remove KolToast from designer entirely Rename component Fix typo Replace remaining KOL-TOAST usages Update desy themes Add readme changes Fix bzst toast theme Remove more unused code Remove unused exports Add missing theme changes Toast Service: Convert Toast component to internal functional component and allow render function instead of description string
label: string; | ||
accessKey: string; | ||
}; | ||
export const InternalUnderlinedAccessKey = ({ label, accessKey }: Props) => { |
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.
Was passiert, wenn der Access-Key nicht im Label-Text enthalten ist?
@@ -75,8 +93,14 @@ export class KolSpanWc implements API { | |||
_hideLabel: false, | |||
_icons: {}, | |||
_label: '', // ⚠ required | |||
_accessKey: '', |
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.
Sollten wir undefined
lassen?!
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.
erledigt
@@ -163,11 +169,17 @@ export class KolTooltip implements API { | |||
@Prop() public _label!: LabelPropType; | |||
|
|||
@State() public state: States = { | |||
_accessKey: '', |
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.
Sollten wir undefined
lassen?!
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.
erledigt
No description provided.