Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

calendar 'select' event missed during 2.0 merge

#354

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/...

Reported by Sergey Avseyev · September 18th, 2010 @ 10:14 AM

State: resolved
Milestone: 2.0.0 Release
Assigned to: MadRabbit MadRabbit

Activity

  1. Sergey Avseyev
  2. Sergey Avseyev
    Sergey Avseyev

    Sorry, I mixed up events names, but bug is still there.

    September 18th, 2010 @ 10:20 AM

  3. MadRabbit
    MadRabbit

    Да, в календарике событие было переименовано немного. Просто прочие виджеты использовали имя change для событий когда пользователь меняет значение, а календарик использовал select. И чтобы сделать все виджеты более менее похожими друг на друга, я переименовал событие.

    September 18th, 2010 @ 10:20 AM

  4. Sergey Avseyev
    Sergey Avseyev

    тогда наверное надо и ивенты переименовать, а то они не срабатывают

    offtopic: тут можно по русски писать?

    September 18th, 2010 @ 11:02 AM

  5. MadRabbit
    MadRabbit
    • 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...

    может что-то вот тут?

    September 18th, 2010 @ 11:12 AM

  6. Sergey Avseyev
    Sergey Avseyev

    Я как-то криво собрал билд, у меня там просто разные ивенты.

    А сейчас 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)
    

    September 18th, 2010 @ 11:45 AM

  7. Sergey Avseyev
    Sergey Avseyev

    ui собирается :) постараюсь воздержаться от опрометчивых высказываний

    September 18th, 2010 @ 11:49 AM

  8. MadRabbit
    MadRabbit

    да ладно. все ошибаются. а вы чего, под рутом чтоли сидите?

    September 18th, 2010 @ 11:54 AM

  9. MadRabbit
    MadRabbit

    я правильно понимаю что тикет можно закрывать?

    September 18th, 2010 @ 12:36 PM

  10. Sergey Avseyev
    Sergey Avseyev

    Да, под рутом :) там удобнее, это немного экономит время.

    По поводу тикета, то можно закрывать. Спасибо

    September 18th, 2010 @ 07:24 PM

  11. MadRabbit
    MadRabbit
    • State changed from new to resolved

    аяяй вас покарает 8)

    найдете что-нибудь еще, приходите починим непременно

    September 18th, 2010 @ 07:46 PM

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.