-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ep/feature/tickets list UI #142
base: ep/feature/refactoringHelpyPsd
Are you sure you want to change the base?
Ep/feature/tickets list UI #142
Conversation
val toolbarFilter = findViewById<ImageButton>(R.id.psd_toolbar_filter_ib) | ||
val toolbarQr = findViewById<ImageButton>(R.id.psd_toolbar_qr_ib) | ||
val ticketsRv = findViewById<RecyclerView>(R.id.tickets_rv) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай binding завезем
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
сделала
|
||
override fun finish() { | ||
super.finish() | ||
PyrusServiceDesk.onServiceDeskStop() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не очень понял смылс этого
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не нужно, удалила
|
||
/** | ||
* Available types of entries. Used for speed optimization to avoid class casting. | ||
* TODO check the possibility of using sealed classes for this purpose. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Да, проще будет sealed классами
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Или можно взять наш кастомный адаптер из основного проекта
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
пока тоже нет необходимости, вообще класс лишний
|
||
private var unreadCount = 0 | ||
|
||
private companion object { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
companion object
Размещаем внизу
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
конкретно здесь пока вообще не нужен удалила
No description provided.