Opened 16 years ago

Closed 15 years ago

Last modified 15 years ago

#1671 closed bug (wontfix)

Calendar - follow naming conventions

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

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 (3)

comment:1 Changed 16 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 15 years ago by paul

Resolution: wontfix
Status: newclosed

comment:3 Changed 15 years ago by (none)

Milestone: 1.2.2

Milestone 1.2.2 deleted

Note: See TracTickets for help on using tickets.