Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change trigger fixes #24

Merged
merged 8 commits into from Jan 22, 2013
Merged

Change trigger fixes #24

merged 8 commits into from Jan 22, 2013

Conversation

@disolovyov
Copy link
Contributor

@disolovyov disolovyov commented Jan 21, 2013

No description provided.

disolovyov
Copy link
Contributor

disolovyov commented on 7ea71af Jan 21, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот этот метод у нас сейчас используется в двух случаях:

  • После загрузки (когда значение ещё не установлено)
  • При установке значения пользователем, только из reloadInfo

Во втором случае не должно быть повторного change, если пользователь правильно всё прописал.

disolovyov
Copy link
Contributor

disolovyov commented on 7ea71af Jan 21, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот это вытащить бы как-то, но оно асинхронно.

valyagolev
Copy link
Contributor

valyagolev commented on 7ea71af Jan 21, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

может нам сделать промиз @currentFileInfo ?

@disolovyov
Copy link
Contributor Author

@disolovyov disolovyov commented Jan 21, 2013

@va1en0k bump

@valyagolev
Copy link
Contributor

@valyagolev valyagolev commented Jan 21, 2013

это прям всерьез работает? я просто не очень понимаю че происходит )

@disolovyov
Copy link
Contributor Author

@disolovyov disolovyov commented Jan 21, 2013

Ну вроде работает. %)

Ещё упростил!

@valyagolev
Copy link
Contributor

@valyagolev valyagolev commented Jan 21, 2013

я правильно понял, что у нас больше нет метода .setValue и вместо него надо мммммм я так понимаю что надо вручную ставить валуе у виджета а потом вручную триггерить change?

@disolovyov
Copy link
Contributor Author

@disolovyov disolovyov commented Jan 21, 2013

Yes. Потому что change не всегда надо триггерить.

@valyagolev
Copy link
Contributor

@valyagolev valyagolev commented Jan 21, 2013

просто обычно элементы триггерят события чтобы что-то произошло во внешнем мире, а тут наоборот – внешнему миру надо триггерить события, чтобы что-то произошло внутри виджета (%


reloadInfo: ->
id = utils.uuidRegex.exec @element.val()
id = if id then id[0] else null

if @currentId != id
if !id || @currentId != id
Copy link
Contributor

@valyagolev valyagolev Jan 21, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тут начинается бредовая логика. "если не айди, или куррент айди не айди, а потом если айди"

Copy link
Contributor

@valyagolev valyagolev Jan 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

серьезно

Copy link
Contributor Author

@disolovyov disolovyov Jan 22, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

“Если id нет или отличается от последнего”

У меня больше нет идей.

@disolovyov
Copy link
Contributor Author

@disolovyov disolovyov commented Jan 22, 2013

@va1en0k It works.

disolovyov added a commit that referenced this issue Jan 22, 2013
@disolovyov disolovyov merged commit 28c1185 into master Jan 22, 2013
@disolovyov disolovyov deleted the change-fixes branch Jan 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants