E
Предлагаю челендж, кто найдет больше какашек в этом коде и предложит как делать правильнее
Нет варианта правильно. Просто его можно менять. Например зачем вообще keys map join - если все можно захардкодить в строку и не выдумывать?
Size: a a a
2020 February 28
N
Evgen
Нет варианта правильно. Просто его можно менять. Например зачем вообще keys map join - если все можно захардкодить в строку и не выдумывать?
чтобы проще и нагляднее было ей управлять?
E
чтобы проще и нагляднее было ей управлять?
В строке прям сложно что-то менять?
N
Evgen
В строке прям сложно что-то менять?
а как тебе такой вариант? https://github.com/NikolayRys/Likely/blob/master/source/dom.js#L109
N
типа join это ненужная сложность и доп обработки?
E
E
типа join это ненужная сложность и доп обработки?
Ну она не нужна
E
в данной ситуации
N
Модалка жеж
где это модалка
N
никакая не модалка
N
используй айфрейм
E
используй айфрейм
Зачем?
N
затем что это правильно в 2020
E
затем что это правильно в 2020
где это правило прописанно?
N
затем что это правильно в 2020
iframe не используется для oAuth
N
iframe не используется для oAuth
с тобой fb, vk, ozon, amazon и прочие поспорят
N
авторизацию через айфрейм делают все, у кого есть единый центр авторизации для разных сервисов
N
не прям все конечно, но многие
E
Чем тебе window.open не угодил?
N
меня больше интересует как он собирается вернуть статус залогинен/не залогинен в изначальное окно