Skip to content
This repository has been archived by the owner on Jul 17, 2024. It is now read-only.

Tapping on time rows in day view throws JS error #27

Open
ajinkyac opened this issue Aug 6, 2013 · 1 comment
Open

Tapping on time rows in day view throws JS error #27

ajinkyac opened this issue Aug 6, 2013 · 1 comment

Comments

@ajinkyac
Copy link

ajinkyac commented Aug 6, 2013

Steps to reproduce:

  1. Invoke the calendar
  2. Switch to day view
  3. Tap on any time rows

It throws error in onTimeSlotTap, Line# 593
When we call this.getCellDate(target) it fails.
As per the code in getCellDate, we fetch the attribute - datetime from 'td' but in our case the 'td' does not have datetime attribute hence returning undefined.

Either we could do (in getCellDate):
var date = dateCell.dom.getAttribute('datetime') || dateCell.up('tr').down('td').getAttribute('datetime');

or on line# 597:

@Stuart98
Copy link
Contributor

I can't replicate this issue with the current version on the example apps. Can you check your using the latest code or put up a test case showing it happening?

Thanks!

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

No branches or pull requests

2 participants