Bug Tracker

Modify

Ticket #1671 (closed bug: wontfix)

Opened 6 years ago

Last modified 6 years ago

Calendar - follow naming conventions

Reported by: dmitrig01 Owned by:
Priority: major Milestone: 1.2.2
Component: ui Version: 1.2.1
Keywords: calendar, names, themes, conventions Cc:
Blocking: Blocked by:

Description

I see this all over the place - <a class="calendar_close" onclick="popUpCal.hideCalendar(0);">Close</a>

That should be <a class="ui-calendar-close">Close</a>, and the event should be bound with a $().bind('click', function(){...});

Change History

comment:1 Changed 6 years ago by iMarc

That is how it was coded initially, but we realized huge performance increases by simply setting the events inline.

comment:2 Changed 6 years ago by paul

  • Status changed from new to closed
  • Resolution set to wontfix

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.