Skip to content

Убрал верстку через xib#113

Open
alexfilimon wants to merge 3 commits into
chausovSurfStudio:masterfrom
alexfilimon:layout-without-xib
Open

Убрал верстку через xib#113
alexfilimon wants to merge 3 commits into
chausovSurfStudio:masterfrom
alexfilimon:layout-without-xib

Conversation

@alexfilimon

Copy link
Copy Markdown
Collaborator

Что сделано:

  • добавил LayoutService для TextField/TextView и дефолтные константы
  • убрал все xib-ы из полей (настоящие мужики верстают из кода 💪)

На что обратить внимание:

  • обратить внимание все ли в документации поправил/добавил
  • немного подправил метод updateViewHeight в TextField-е - для того, чтобы поле не прыгало для дефолтного layout-а

PS:

  • документации слишком много и ее поддерживать оч сложно, особенно когда понимаешь что одно ссылается на другое и так оч далеко 😭
  • документация не отражает код - произошел рассинхрон, было бы неплохо проверить всю документацию

@alexfilimon alexfilimon self-assigned this Apr 10, 2021
@alexfilimon alexfilimon changed the title Убрал верстку через xib [after #112] Убрал верстку через xib Apr 10, 2021
@chausovSurfStudio

Copy link
Copy Markdown
Owner

да я документацию в том виде которая сейчас вообще планирую выпилить) потому на нее особо смотреть не буду
ибо да, ее дохрена, и сути она не отражает.

планирую лучше более детально описать все аспекты в коде, чтобы автоматически генерируемая документация была симпотной, ну и чтобы при работе можно было все эти детали увидеть(
А уже здест при проекте оставить только объяснение каких-то скрытых деталей, как с этим работать и жить и тд

@chausovSurfStudio

Copy link
Copy Markdown
Owner

что-то тут ругаются на тебя

Снимок экрана 2021-04-23 в 20 03 25

@alexfilimon

Copy link
Copy Markdown
Collaborator Author

да я документацию в том виде которая сейчас вообще планирую выпилить) потому на нее особо смотреть не буду
ибо да, ее дохрена, и сути она не отражает.

планирую лучше более детально описать все аспекты в коде, чтобы автоматически генерируемая документация была симпотной, ну и чтобы при работе можно было все эти детали увидеть(
А уже здест при проекте оставить только объяснение каких-то скрытых деталей, как с этим работать и жить и тд

Согласен. Норм тема!
А то текущая документация это неподдерживаемая куча текста

@alexfilimon

Copy link
Copy Markdown
Collaborator Author

что-то тут ругаются на тебя

Снимок экрана 2021-04-23 в 20 03 25

pod install в example проекте надо сделать

@alexfilimon alexfilimon changed the title [after #112] Убрал верстку через xib Убрал верстку через xib Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants