Skip to content
This repository has been archived by the owner. It is now read-only.

Bug with cell modifier function for week view #424

Closed
Sh1d0w opened this Issue Aug 15, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@Sh1d0w
Copy link

Sh1d0w commented Aug 15, 2016

Bug description

This ticket is regarding a feature implemented a few days ago #418 .

The problem is that in the week view, the slots passed to the function are only for the first day of the week and the rest are not. This way the calendar week view could not be properly painted as not all days/slots are passed to the callback and so the callback function can not apply correct CSS classes to the not available cells.

I guess this is because of the implementation of the current row/col system that is used in the week view? I guess each time slot requires its own cell in order that to work properly.

Versions

Angular: 1.4.9
Calendar directive: 0.23.0

Browser name and version:
Every browser

@Sh1d0w Sh1d0w changed the title Bug with cell modifier function for day view Bug with cell modifier function for week view Aug 15, 2016

@mattlewis92

This comment has been minimized.

Copy link
Owner

mattlewis92 commented Aug 15, 2016

Ah I hadn't considered the week view, it should be relatively easy to fix this with a bit of refactoring.

@mattlewis92 mattlewis92 added the bug label Aug 15, 2016

@mattlewis92 mattlewis92 modified the milestones: 0.23.x, 0.24.x Aug 15, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.