calendar 'select' event missed during 2.0 merge
Reported by Sergey Avseyev | September 18th, 2010 @ 10:14 AM | in 2.0.0 Release
Hi
It seems that calendar triggering calendar event was broken
during last merge.
This code says that there is 'select' event
http://github.com/rightjs/rightjs-ui/blob/4526edad43bc7b668b61/src/...
but this code fires 'change' event instead
http://github.com/rightjs/rightjs-ui/blob/4526edad43bc7b668b61/src/...
Comments and changes to this ticket
-
Sergey Avseyev September 18th, 2010 @ 10:16 AM
- Tag set to calendar
-
Sergey Avseyev September 18th, 2010 @ 10:20 AM
Sorry, I mixed up events names, but bug is still there.
-
MadRabbit September 18th, 2010 @ 10:20 AM
Да, в календарике событие было переименовано немного. Просто прочие виджеты использовали имя
change
для событий когда пользователь меняет значение, а календарик использовалselect
. И чтобы сделать все виджеты более менее похожими друг на друга, я переименовал событие. -
Sergey Avseyev September 18th, 2010 @ 11:02 AM
тогда наверное надо и ивенты переименовать, а то они не срабатывают
offtopic: тут можно по русски писать?
-
MadRabbit September 18th, 2010 @ 11:12 AM
- Milestone set to 2.0.0 Release
- Tag changed from calendar to calendar, ui
- Milestone order changed from 8 to 0
Везде можно.
Как именно не срабатывают?
http://github.com/rightjs/rightjs-ui/blob/master/src/calendar/calen...
может что-то вот тут?
-
Sergey Avseyev September 18th, 2010 @ 11:45 AM
Я как-то криво собрал билд, у меня там просто разные ивенты.
А сейчас ui совсем не собирается, а core пишет ошибку
# rake build (in /root/code/javascript/rightjs-core) * Composing the source file rake aborted! No such file or directory - build/right-src.js /root/code/javascript/rightjs-core/Rakefile:87 (See full trace by running task with --trace)
-
Sergey Avseyev September 18th, 2010 @ 11:49 AM
ui собирается :) постараюсь воздержаться от опрометчивых высказываний
-
MadRabbit September 18th, 2010 @ 11:54 AM
да ладно. все ошибаются. а вы чего, под рутом чтоли сидите?
-
Sergey Avseyev September 18th, 2010 @ 07:24 PM
Да, под рутом :) там удобнее, это немного экономит время.
По поводу тикета, то можно закрывать. Спасибо
-
MadRabbit September 18th, 2010 @ 07:46 PM
- State changed from new to resolved
аяяй вас покарает 8)
найдете что-нибудь еще, приходите починим непременно
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
RightJS Core Tickets