Skip to content

RA-08#6

Merged
KinCadeFoster merged 2 commits intomasterfrom
RA-08
Sep 7, 2024
Merged

RA-08#6
KinCadeFoster merged 2 commits intomasterfrom
RA-08

Conversation

@KinCadeFoster
Copy link
Copy Markdown
Owner

  • Создал файл адаптера CategoriesListAdapter
  • Инициализировал RecyclerView во фрагменте CategoriesListFragment
  • Убедится, что карточка элемента категории в списке кликабельная
  • небольшой рефакторинг

- Создал файл адаптера CategoriesListAdapter
- Инициализировал RecyclerView во фрагменте CategoriesListFragment
- Убедится, что карточка элемента категории в списке кликабельная
- небольшой рефакторинг
@ievetrovdev
Copy link
Copy Markdown

@dimen/padding_10
@dimen/padding_10
В таких случаях в стиле лучше указывать просто padding одной строкой

<item name="textAllCaps">true</item>
</style>

<style name="StyleNavButton" parent="Widget.Material3.Button">
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

выше убрал правильно, а вот здесь не надо. Мы наследуемся от кнопки из materialDesign

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

я тут на баг нарвался, на Honor 200 pro. Почему то цвет не устанавливался (был фиолетовый). Все голову сломал, код перепахал, разработчиков подтянул. В конечном итоге выяснилось что это девайс специфик баг, на Pixel 5 и эмуляторах цвет кнопок корректный. При установке цвета явно в кнопке, все нормально становится, если через стиль, то все так же, цвет текста кнопок не по дизайну. В конечном итоге по результатам теста, parent="Widget.Material3.Button" как будь то ни на что вообще не влияет и поэтому снес.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

В твоём случае да, параметр "Widget.Material3.Button" ставится по дефолту, поскольку базовая тема твоего приложения наследуется от parent="Theme.Material3.DayNight.NoActionBar". У новых приложений это по дефолту установлено, однако у приложений чуть постарше часто бывают свои кастомные темы, или просто используют Material2, в таких случаях лучше прописывать. В целом можно и убрать.

- правки по замечаниям
Copy link
Copy Markdown

@ievetrovdev ievetrovdev left a comment

Choose a reason for hiding this comment

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

Молодец, ставлю апрув. Из коммента не совсем понял, в итоге баг удалось поправить? И в чём баг заключается конкретно: цвет букв или цвет самой кнопки не тот?
Отпиши в личку по этой проблеме, помогу решить, если остался баг

@KinCadeFoster KinCadeFoster merged commit f6126b8 into master Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants