Conversation
…onents into rkaraivanov/date-picker
739aa01 to
4409d54
Compare
…onents into rkaraivanov/date-picker
* Added tests
|
@AnjiManova I found the piece about the end-user experience in the date picker specification in Angular. https://github.com/IgniteUI/igniteui-angular/wiki/Date-Picker-Specification#3-functionality Can you confirm the behavior? |
|
@kdinev, we confirm the behaviour described in the specs: "In dialog mode, the input will always be read-only, and clicking anywhere on it will open the dialog." |
* The date picker input is now readonly in dialog mode. * When the component is in dialog mode, the dialog is shown on clicking the date picker input field as well. * When the picker is shown, regardless of mode, the current selected value/last active date will be focused.
|
@AnjiManova @kdinev |
…onents into rkaraivanov/date-picker
|
@AnjiManova Is it me or the Design Handoff in the spec only deals with the actual calendar popup but nothing on the input or component as a whole. I realize it's mostly the same as any Input with a built-in calendar toggle and clear, but still at least one detail I think we might've missed: |
| ` | ||
| : html` | ||
| <igc-dialog | ||
| aria-label="Select date" |
There was a problem hiding this comment.
This should ideally be localized :)
Perhaps, we can re-use the same string from the resource strings at least, seems like it'll make some sense.
| month, month-inner, year, year-inner, selected, current" | ||
| > | ||
| ${!this.isDropDown | ||
| ? html`<slot name="title" slot="title">Select date</slot> ` |
There was a problem hiding this comment.
This is already the default title from the resource strings from what I can see, so passing this slot along... only disables the resource string handling? I think should be removed if so.
…onents into rkaraivanov/date-picker
@damyanpetev, no, it's not you; in the handoff, the main focus was on the popup, and yes, you are right - the date picker compound parts are the popup and the input with a built-in toggle and clear icon. Unfortunately, we don't have visual examples for the input, which we should update and provide to make it clearer. Regarding your question, the variants provided in the kit currently start with a toggle because in AB, the default and only view of the component is with a toggle at the start. This is why we assumed that we are going to cover only one toggle position for the component for now. If there is a way to expose a property for the toggle icon in AB where you can select the toggle position, we will add such a variant in our Figma kits. In addition, if we want the default position of the toggle to be at the end, that should also be changed in AB. In terms of your question whether there is a need to have the start and end positions of the toggle icon, IMO, yes, the current behaviour of the component you have described (allowed the toggle to be overridden and move from start to end) should stay the same, and we could consider supporting the two options (again AB should replicate the kits variants and after that we will make the change). |
@AnjiManova 👍 |
damyanpetev
left a comment
There was a problem hiding this comment.
The current state LGTM and covers the designs we need.
@rkaraivanov from the discussion for the toggle position, leaving to you if you want that to be logged separately so this can be merged.
|
Yeah, this would be a nice feature for components that have a toggle action part. @AnjiManova |
Closes #174