Skip to content

Conversation

@ivan-magda
Copy link
Member

@ivan-magda ivan-magda commented Aug 29, 2018

Задача: #2008, #2005, #2039

Описание:

  • Обновили внешний вид экрана обучения и тренировки.
  • Поддержали Swift 4.1.

Теперь на экранах обучения и тренировки списки в виде карточек, как было в макете.

ezgif com-resize

@ivan-magda ivan-magda self-assigned this Aug 29, 2018
@kvld kvld changed the title Тew training and learning tab screens New training and learning tab screens Aug 30, 2018
@Ostrenkiy
Copy link
Contributor

Но баги надо бы все равно пофиксить

@ivan-magda
Copy link
Member Author

Да, я это начал делать вчера, в отдельной ветке, чтобы вас уведомлениями не тревожить.

Copy link
Contributor

@kvld kvld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно мерджить

"images" : [
{
"idiom" : "universal",
"filename" : "clipboard-checklist.png",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оставляем конечно, НО. Какая причина, чтобы не юзать вектор?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я как-то этот API пропустил для себя и по стариночки клипаю изображения необходимого размера.
Нужно будет разобраться)

let description: String
let progress: Float
var type: TopicType
/// In minutes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(:

TimeInterval

A number of seconds.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лол, конечно :trollface:

@ivan-magda ivan-magda merged commit 1d09cb7 into dev Sep 20, 2018
@ivan-magda ivan-magda deleted the feature/exam-new-training-tab-screen branch September 20, 2018 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants