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

[ns.Box] Динамическая вставка нод в боксе в неправильном порядке #614

Closed
vitkarpov opened this issue Jul 14, 2016 · 7 comments

Comments

@vitkarpov
Copy link
Member

Лейаут:

'wrapper@': function() {
  return {
    v1: true,
    v2: true
  }
}

Обычно в боксе появляются ноды в том же порядке, в каком они были объявлены в лейауте в объекте, однако, бывает так, что сперва идет v2.

Кажется, дело в том, что мы итерируемся по ключам в объекте(лейаут бокса) и показываем/скрываем ноды — однако, for..in по ключам не гарантирует порядок (хотя в большинстве случаев и ок).

@chestozo
Copy link
Member

Что-то мне кажется я такое делал и даже тесты писал про то, что виды должны быть в правильном порядке, помню помню.
А тест сможешь забубенить?

@vitkarpov
Copy link
Member Author

Что-то мне кажется я такое делал

А удалось воспроизвести в тесте, в итоге? Мы просто видели такое в приложении, но воспроизводится очень редко.

@chestozo
Copy link
Member

Вот, нашёл - фиксилась последовательность видов внутри бокса #499

@chestozo
Copy link
Member

#499 #502

@vitkarpov
Copy link
Member Author

Хм. Судя по всему, теперь должно работать как надо.

@chestozo
Copy link
Member

По идее так уже довольно давно работает (с Feb 3, 2015), а у тебя проблемы давно эти были?

@vitkarpov
Copy link
Member Author

Да, скорее всего, я нашел какой-то старый TODO в коде просто.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants