Size: a a a

Cicerone Chat (RUS)

2018 October 04

I

Ilya in Cicerone Chat (RUS)
а ну то есть на подобии того что раньше использовалось - startActivityForResult и туда бросать результат в зависимости от которого закрывать вторую активти?
источник

ES

Eugene Shapovalov in Cicerone Chat (RUS)
Ilya
а ну то есть на подобии того что раньше использовалось - startActivityForResult и туда бросать результат в зависимости от которого закрывать вторую активти?
нет. Intent.FLAG_ACTIVITY_CLEAR_TOP
источник

I

Ilya in Cicerone Chat (RUS)
Eugene Shapovalov
нет. Intent.FLAG_ACTIVITY_CLEAR_TOP
тут другое. Тут есть 2 варианта, из активти3 по нажатию назад я должен провалиться на активити2, а при нажатии спец кнопки на активити1, а с этим флагом я всегда буду мимо активити2 пролетать
источник

I

Ilya in Cicerone Chat (RUS)
ну по итогам сделал через router.sendActivityResult
источник
2018 October 05

ИИ

Иван Иванов in Cicerone Chat (RUS)
Добрый день всем. Прошу подсказать с навигаецией. В приложении есть 2 активити(до логина в приложение и после), activity1 может вызвать activity2 и наоборот. В activity1 есть контейнер и фрагменты, просто меняю их в контейнере. В activity2 тоже контейнер и другие фрагменты между которыйми происходит навигация. Правильно ли я понял, что мне нужно создавать 2 екземпляра роутера(первый для activity1, второй для activity2)? Или лучше отказаться от двух активити и сделать все в одной?
источник

ИИ

Иван Иванов in Cicerone Chat (RUS)
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
Лучше конечно в одной активити. А вместо ваших активити - фрагменты контейнеры
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
Я мигрировал гитфокс на новую версию Чичероне:
https://gitlab.com/terrakok/gitlab-client

можно ознакомиться с новым подходом к описанию экранов
источник
2018 October 06

AP

Alexey Pushkarev in Cicerone Chat (RUS)
Konstantin Tskhovrebov
Если кто-то внесет отдельным модулем и в мр опишет, зачем это надо и как использовать. То приму
Я Вам пулл реквест на это дело завез, рассмотрите?
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
когда появится возможность - обязательно
источник
2018 October 07

AP

Alexey Pushkarev in Cicerone Chat (RUS)
Ок, спасибо
источник
2018 October 09

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
Alexey Pushkarev
Ок, спасибо
такое впечатление, что это отписка преподу на экзамене.
хоть сами бы проверили код перед тем как пушить в общую либу 😐
и имена одного и того же разные в одинаковых фрагментах
и несоблюдение общего стиля
и вообще критикал баги в поведении навигатора!!!

вы разбирались как Чичероне работает? или просто хотели приобщиться к проекту?
пожалуйста, проявите уважение к проекту! каждый мр проверьте и вымучайте на все кейсы!
вы видели спеку команды бекТу?

пока МР - ломает все, глупо показывая, что "вот, удалось вызвать команду на aac навигаторе". код на коленке! такие нельзя предлагать как МР!

кроме того, есть ОЧЕНЬ важный пункт при использовании такого навигатора: надо четко описать, что используя команды Чичероне нельзя описывать поведение переходов на xml графе!
всякие app:popUpToInclusive="false"
это легко все сломает!
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
если вы серьезно хотите сделать такой навигатор, то опишите все возможные кейсы. напишите доку. как, почему и зачем надо использовать ваш навигатор. какие ограничения. как правильно совместить его с xml разметкой переходов.
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
сейчас это "я сделяль" 😣
источник

ES

Eugene Shapovalov in Cicerone Chat (RUS)
@terrakok
А вот у меня вопрос возник. В модуле aac_navigation library должна подключаться как api, чтобы транзитивно подключать и саму Cicerone?
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
Konstantin Tskhovrebov
такое впечатление, что это отписка преподу на экзамене.
хоть сами бы проверили код перед тем как пушить в общую либу 😐
и имена одного и того же разные в одинаковых фрагментах
и несоблюдение общего стиля
и вообще критикал баги в поведении навигатора!!!

вы разбирались как Чичероне работает? или просто хотели приобщиться к проекту?
пожалуйста, проявите уважение к проекту! каждый мр проверьте и вымучайте на все кейсы!
вы видели спеку команды бекТу?

пока МР - ломает все, глупо показывая, что "вот, удалось вызвать команду на aac навигаторе". код на коленке! такие нельзя предлагать как МР!

кроме того, есть ОЧЕНЬ важный пункт при использовании такого навигатора: надо четко описать, что используя команды Чичероне нельзя описывать поведение переходов на xml графе!
всякие app:popUpToInclusive="false"
это легко все сломает!
"имена одного и того же разные в одинаковых фрагментах" - если вы про btn_* , то да есть такое. Но ведь нигде не написаны требования к кода стайлу, тогда надо какой-то contrubution guide заводить.

"вообще критикал баги в поведении навигатора!!!" - Если вы про асинхронные транзакции фрагмент менеджера, то  исправим, ок.

"вы разбирались как Чичероне работает? или просто хотели приобщиться к проекту?" - Если бы не разобрался, не писал бы навигатор )) Про приобщиться к проекту не понял вопроса. Можно также сделать отдельный репозиторий с расширением для cicerone и паблишить на мавен как отдельный артефакт, но логичнее конечно в основном репозитории как модуль.

"пожалуйста, проявите уважение к проекту! " - пожалуйста, проявляйте уважение к контрибьютерам и пишите ТОЛЬКО конструктивные замечания, без перехода на личности.

"кроме того, есть ОЧЕНЬ важный пункт при использовании такого навигатора: надо четко описать, что используя команды Чичероне нельзя описывать поведение переходов на xml графе!" - а смысл тогда от навигатора? Только в том, чтобы визуально граф перехода видеть?

"всякие app:popUpToInclusive="false"
это легко все сломает!" - через команду forward, что в итоге вызывает navigate у navcontroller работает. Понятно, что если пользователю либы позволить всякие таки флаги, то он может что-нибудь нехорошее натворить. Ну тут я согласен, надо подумать что какие флаги можно использовать, а какие не стоит.

Про доку согласен, надо сделать ее в markdown как на главной странице.
источник

KT

Konstantin Tskhovrebov in Cicerone Chat (RUS)
Когда стиль не описан, это не значит, что модно писать как попало, это значит, что надо самому изучать код на предмет стиля.
По работе чичероне: самая важная часть - это команды, у каждой из которых есть четкая семантика. А у вас в трех местах нарушение: реплейс неправильно реализован. Бекту неправильно реализован. В xml добавлены свойства перехода, которые противоречат ожиданиями библиотеки.

Я не перехожу на личности. Я любому с таким pr написал бы тоже самое. Либо просто закрыл pr.

"Зачем такой навигатор нужен?" Это главный вопрос. Пока он нужен только для отображения графа.
источник

AP

Alexey Pushkarev in Cicerone Chat (RUS)
ok, поправим.
источник

ns

natan solnysh in Cicerone Chat (RUS)
Moxy+Cicerone.есть активити-контейнер и  фрагмент.пытаюсь сделать фрагмент корневым экраном вызывая newRootScreen,если это делаю в в презентере onFirstViewAttach то соответственно при перезаходе в приложение,пустой контейнер,если в activity onCreate то  при смене ориентации  пустой фрагмент,как сделать фрагмент корневым?
источник

NY

Nikita Yatskivskiy in Cicerone Chat (RUS)
Konstantin Tskhovrebov
Лучше конечно в одной активити. А вместо ваших активити - фрагменты контейнеры
Извините, а почему использование одной активити всё же предпочтительнее? Единообразие? Или по другим причинам?
источник