umnik
/
YAF
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Labels Milestones

simplified_code #10

Merged
hesherbombom merged 4 commits from simplified_code into master 3 months ago
Conversation 0 Commits 4 Files Changed 2
umnik commented 3 months ago
Owner

Упрощение кода

  1. Убран метод показа тоста. Всё же правильнее из тестов не показывать юайные штуки, тем более тосты считаются устаревшими
  2. Упрощён код для прокрутки до нужной вьюхи. Вместо добавления перехватов ещё нескольких исключений (в данном случае - кнопка видна на 30% была), просто пытаемся крутить дальше. Если приложение развалилось, то прокрутка и так ничего не сделает и свалимся уже нормально
Упрощение кода 1. Убран метод показа тоста. Всё же правильнее из тестов не показывать юайные штуки, тем более тосты считаются устаревшими 1. Упрощён код для прокрутки до нужной вьюхи. Вместо добавления перехватов ещё нескольких исключений (в данном случае - кнопка видна на 30% была), просто пытаемся крутить дальше. Если приложение развалилось, то прокрутка и так ничего не сделает и свалимся уже нормально
umnik requested review from hesherbombom 3 months ago
hesherbombom was assigned by umnik 3 months ago
hesherbombom reviewed 3 months ago
app/build.gradle Outdated
@ -44,3 +44,3 @@
implementation 'org.jetbrains.kotlinx:kotlinx-coroutines-android:1.6.4'
// Тесты
// ТестыTOsUiAction.kt
hesherbombom commented 3 months ago
Collaborator

Так и запланировано?

Так и запланировано?
umnik commented 3 months ago
Poster
Owner

Кек. Видимо курсор тут был, когда из буфера обмена вставку делал

Кек. Видимо курсор тут был, когда из буфера обмена вставку делал
umnik commented 3 months ago
Poster
Owner

Кек. Видимо курсор тут был, когда из буфера обмена вставку делал

Кек. Видимо курсор тут был, когда из буфера обмена вставку делал
umnik marked this conversation as resolved
hesherbombom merged commit a0a814ae6e into master 3 months ago
hesherbombom deleted branch simplified_code 3 months ago

Reviewers

hesherbombom was requested for review 3 months ago
The pull request has been merged as a0a814ae6e.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
hesherbombom
Labels
Apply labels
Clear labels   
bug

Something is not working   
duplicate

This issue or pull request already exists   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
question

More information is needed   
wontfix

This won't be fixed
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
hesherbombom
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: umnik/YAF#10
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'simplified_code'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes