Nextcloud-vue: Datepicker to narrow when show-week-numbers

[raimund-schluessler]

The datepicker overflows when week-numbers are shown. Set :show-week-numbers="true" in the documentation to reproduce:

Happens in Calendar 2.0.3 now. @georgehrke @skjnldsv

[skjnldsv]

Should be fixed already with https://github.com/nextcloud/nextcloud-vue/pull/995

[skjnldsv]

https://github.com/nextcloud/nextcloud-vue/pull/995#issuecomment-610216841

[skjnldsv]

Yep, perfect for me in https://nextcloud-vue-components.netlify.com/#/Components/DatetimePicker

[skjnldsv]

Aaah, ok, I fixed it for type="week", not show-week-numbers

[georgehrke]

I updated to 1.5.0 before releasing 2.0.3

[skjnldsv]

Well, I cannot reproduce with show-week-numbers="true" on the documentation, what template code are you using?

[georgehrke]

https://github.com/nextcloud/calendar/blob/master/src/components/Shared/DatePicker.vue#L24

[georgehrke]

Well, I cannot reproduce with show-week-numbers="true" on the documentation, what template code are you using?

It's show-week-number="true", not show-week-numbers="true"

[georgehrke]

[raimund-schluessler]

Still an issue in the latest version, i.e. the Calendar app.

[raimund-schluessler]

The problem is this line https://github.com/nextcloud/nextcloud-vue/blob/master/src/components/DatetimePicker/index.scss#L63 which doesn't take the week number column into account. But I don't know how to overwrite it, because there is no class indicating if the week numbers are shown. @skjnldsv Any idea? Maybe I am missing something.

[raimund-schluessler]

I added a feature request to the datepicker library to have a class set when show-week-number is enabled: https://github.com/mengxiong10/vue2-datepicker/issues/600

[raimund-schluessler]

Fix is in #1917. Adjustments to the datepicker library were not necessary.