search

sadr0b0t / yashlang

PeerTube and YouTube player for Android with local playlists and whitelisted recommendations
GNU General Public License v3.0
51 stars 3 forks source link

Перевести все экраны в режим стейт-машин #52

Closed sadr0b0t closed 3 years ago

sadr0b0t commented 3 years ago

Рефактор: перевести код всех экранов, у которых элементы управления появляются и исчезают в зависимости от динамических событий и от контекста, в режим стейт-машины в стиле React (завести переменные-флаги и енумы для текущих состояний, элементы управления настраивать в едином методе updateControlsVisibility).

Сделал так здесь: https://github.com/sadr0b0t/yashlang/blob/master/app/src/main/java/su/sadrobot/yashlang/ConfigurePlaylistsNewItemsFragment.java

И еще частично в WatchVideoActivity

sadr0b0t commented 3 years ago

ViewPlaylistNewItemsFragment: https://github.com/sadr0b0t/yashlang/commit/b4a42519f5a3b2f481f448831659d55b9e52c902

sadr0b0t commented 3 years ago

AddPlaylistActivity.java и AddRecommendedPlaylistsActivity.java https://github.com/sadr0b0t/yashlang/commit/622254820af7115d53ac19fc22a2cb0496c45cdf

sadr0b0t commented 3 years ago

В History и Starred еще есть переключение панелек списка и пустого списка, но это совсем просто, можно обойтись без стейт-машины пока.