-
Notifications
You must be signed in to change notification settings - Fork 888
feat(Calendar): add week numbers #4555
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: v3
Are you sure you want to change the base?
Conversation
I'm not sure if we truly want these, but I've made them just in case, I only undestand cs, sk and en so the rest are AI translations... I did some research and I'm not sure if we really need this header as it is not space efficient and most users know that the leftmost column in calendar is used for week numbers.
I'm not sure if we want the header for the week number column, so I put it inside a separate commit to make it easier to remove if necessary. We need to wait for the next RekaUI release where the |
commit: |
test/components/Calendar.spec.ts
Outdated
['without fixedWeeks', { props: { fixedWeeks: false } }], | ||
['without monthControls', { props: { monthControls: false } }], | ||
['without yearControls', { props: { yearControls: false } }], | ||
['without weekNumbers', { props: { weekNumbers: true } }], |
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.
@J-Michalek This is wrong as the title should be "with weekNumbers".
β¦dar-week-numbers
Shall we close this to take advantage of unovue/reka-ui#1959 instead? |
We can leave this open, it just needs some finishing touches now that the utility function is released. I will look into this this week for sure! |
π Linked issue
Resolves: #4231
β Type of change
π Description
I've added the week numbers to the leftmost column of the calendar.
π Checklist