#354 ✓resolved
Sergey Avseyev

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
  • Sergey Avseyev

    Sergey Avseyev September 18th, 2010 @ 10:20 AM

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

  • MadRabbit

    MadRabbit September 18th, 2010 @ 10:20 AM

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

  • Sergey Avseyev

    Sergey Avseyev September 18th, 2010 @ 11:02 AM

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

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

  • MadRabbit

    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

    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

    Sergey Avseyev September 18th, 2010 @ 11:49 AM

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

  • MadRabbit

    MadRabbit September 18th, 2010 @ 11:54 AM

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

  • MadRabbit

    MadRabbit September 18th, 2010 @ 12:36 PM

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

  • Sergey Avseyev

    Sergey Avseyev September 18th, 2010 @ 07:24 PM

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

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

  • MadRabbit

    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.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

RightJS Core Tickets

Shared Ticket Bins

People watching this ticket

Tags

Pages