Skip to content
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

Cannot detect mouse over of a background event in month view if another background event is on the same row #4750

Closed
laoneo opened this issue Jun 26, 2019 · 7 comments

Comments

@laoneo
Copy link
Contributor

laoneo commented Jun 26, 2019

Bug Reports

I'm using tippy as tooltip library. So far it works. But when I do rendering as background thourgh the even objects properties, then it renders only the last element. This due the fact that per event one element is rendered above each other.

I'v opened an issue on tippy itself but the maintainer there reported back that some listeners do need to be reapplied. Here you can see the full report atomiks/tippyjs#529.

I'v prepared a code pen where you can see the issue https://codepen.io/anon/pen/rEGaZo. The problem is that when hovering over the second event, the tooltip doesn't show up. Is there something can be done here in fullcalendar?

@acerix acerix changed the title Overlays issue when rendering is background Cannot detect mouse over of a background event in month view if another background event is on the same row Jun 26, 2019
@acerix
Copy link
Member

acerix commented Jun 26, 2019

I can see the second background event adds a "skeleton" over the entire row (week) which prevents detecting when the mouse is over the first background event since it's covered.

Setting "pointer-events: none;" on the skeleton seems to work:

https://codepen.io/acerix/pen/dBVjrQ?editors=0111

@laoneo
Copy link
Contributor Author

laoneo commented Jun 27, 2019

Thanks for confirmation. Are you guys going to fix it or should I do a pr or is it expected that the users should add the remaining CSS to their app?

@arshaw
Copy link
Member

arshaw commented Jul 25, 2019

we decided to do background event rendering like this b/c it plays better with print-mode, but we might walk that back decision (might refactor how event DOM/CSS works). Until then, @acerix 's workaround is your best bet.

@arshaw arshaw added this to the next-minor milestone Jul 25, 2019
@arshaw
Copy link
Member

arshaw commented Jul 25, 2019

To fix this more quickly, I'll force FullCalendar to apply pointer-events:none on these dummy cells that are blocking the real events.

@arshaw
Copy link
Member

arshaw commented Aug 10, 2019

This was fixed in 4.3.0 though please start using 4.3.1
Releases: https://github.com/fullcalendar/fullcalendar/releases

@laoneo can you confirm that things are all good?

@laoneo
Copy link
Contributor Author

laoneo commented Aug 12, 2019

Yes, it works. Just wondering if it could be solved with changes in the CSS file, instead of hardcoding directly into the HTML code within the JS file.

@acerix
Copy link
Member

acerix commented Oct 28, 2019

I had to come back to this for a related issue and noticed the new release doesn't fix my codepen so there might still be an issue:

https://codepen.io/acerix/pen/abbyygN?editors=0110

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants