Насчёт первого пункта я пересмотрю, но насчёт второго если честно не понял замечания то есть как я понял AboutListItemViewModel быть вообще не должно?
я же получаются из xml ссылаюсь так раз таки на эти значения.
согласен, геттеры должны быть. но они должны возвращать не значения, а какой-то обзервабл. LiveData или из рх.
суть в том, что вью модель хранит стейт и вьюмодель изменяет его. в вью на эти изменения подписывается.
а если вью просто надо прочитать значения один раз это ок, но в более сложных случаях начнется хаос и боль.