-
Notifications
You must be signed in to change notification settings - Fork 272
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
feat(ui5-calendar-legend): introduce ui5-calendar-legend component #7706
Conversation
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.
In openui5 you can reach the calendar legend with tab, but here it's not possible.
In ui5 tabchain is: month , year, calendar and finishes with legend
Here tabchain is: calendar, month, year (legend is inclued if you click on some item inside it and then the chain is calendar, month, year, legend)
Also please change all since tag that remain to version 1.23.0
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.
For me it looks good. Please request review from the team again
…7706) Introducing the new CalendarLegend, CalendarLegendItem, SpecialCalendarDate Web Components. The Calendar Legend has four properties: *hideToday (hide-today) - Hides the 'Today' item in the Calendar Legend; *hideSelectedDay (hide-selected-day) - Hides the 'Selected' item in the Calendar Legend; *hideWorkingDay (hide-working-day) - Hides the 'Working Day' item in the Calendar Legend; *hideNonWorkingDay (hide-non-working-day) - Hides the 'Non-Working Day' item in the Calendar Legend; Accepting <ui5-calendar-legend-item>; ------- The Calendar Legend Item has two properties: type - Accepting string value of ("Type01"..."Type20"); ------- SpecialCalendarDate accepts the property: type - which should be a string value of ("Type01"..."Type20"); emphasizing the special days in the Calendar.
Is there a particular reason for not having a top-level element in the calendar's shadow root? @hinzzx UI5Element.ts:563 The shadow DOM for ui5-calendar does not have a top level element, the getDomRef() method might not work as expected |
Part 1:
Introducing the
<ui5-calendar-legend>
components as Part 1 of the feature request about having Classic UI5-like Calendar Legend component.Currently our
<ui5-calendar-legend>
component presents only the default 4 types (Today, Selected, Working and NonWorking).It has 4 properties:
hideToday
(hide-today) - Hides the 'Today' item in the Calendar Legend;hideSelectedDay
(hide-selected-day) - Hides the 'Selected' item in the Calendar Legend;hideWorkingDay
(hide-working-day) - Hides the 'Working Day' item in the Calendar Legend;hideNonWorkingDay
(hide-non-working-day) - Hides the 'Non-Working Day' item in the Calendar Legend;Part 2:
Part 3:
<ui5-calendar-legend>
with the<ui5-calendar>
.SpecialCalendarDate
s, emphasizing special days in the Calendar;