2gis / slot

JavaScript фреймворк, позволяющий быстро и просто создавать модульные изоморфные приложения.
http://2gis.github.io/slot
BSD 2-Clause "Simplified" License
86 stars 10 forks source link

Add an opportunity to determine history popstate #246

Closed RrredHead closed 8 years ago

RrredHead commented 8 years ago

Added an opportunity to determine that new state of app obtained with the use of historyApi popstate method.

Trufi commented 8 years ago

А нельзя сделать это более публичным методом, чтобы не писать _.get($appState, 'state._stateFromHistory')?

ovvost commented 8 years ago

Внесу свои 5 копеек: этот флаг вообще относится не ко всему стейту, а к одному конкретному диффу и приписывать его нужно не к стейту а бросать в 'statechange'. Что дальше произойдет с этой информацией - на совести приложения.

ctizen commented 8 years ago

А нельзя отдельно подписаться на popstate там где это нужно?

RrredHead commented 8 years ago

@heilage-nsk Нет, так не получается

ctizen commented 8 years ago

Ну, в текущем решении криминала не вижу.

ctizen commented 8 years ago

Хотя нет, вижу. Может не делать лишних мутаций? В смысле, _.extend({}, state, {...})

RrredHead commented 8 years ago

@Trufi Этот флаг прилетает в diff. Можете его брать оттуда.

RrredHead commented 8 years ago

@heilage-nsk Избавился от одной мутации @ovvost и @ilyapro Переименовал флаг, более нейтральный. Хорошо звучит и со стэйтом и с дифом :-D

ilyapro commented 8 years ago

:+1:

Trufi commented 8 years ago

Да, я имел ввиду что использование state._что-то говорит, что это приватное поле, которое лучше использовать.

Теперь ок :+1: