Skip to content
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

#64: added custom hamburger in toolbar and changed font-family for title #68

Merged
merged 1 commit into from
Feb 12, 2021

Conversation

temnik15
Copy link
Collaborator

@temnik15 temnik15 commented Feb 4, 2021

No description provided.

@temnik15 temnik15 added this to the MainActivity milestone Feb 4, 2021
@temnik15 temnik15 self-assigned this Feb 4, 2021
@temnik15 temnik15 force-pushed the feature/HAWK-64/create-toolbar-and-hamburger branch from ad592aa to 7e2b231 Compare February 4, 2021 12:02
@temnik15 temnik15 force-pushed the feature/HAWK-64/create-toolbar-and-hamburger branch 2 times, most recently from 2394b49 to 744a1d5 Compare February 4, 2021 12:10
@temnik15
Copy link
Collaborator Author

temnik15 commented Feb 4, 2021

Так как drawer выезжает поверх toolbar, предлагаю отказаться от анимации аналогичной ActionBarDrawerToggle, так как указанная view использует DrawerArrowDrawable, которая вручную отрисовывает всю анимацию, то есть для реализации аналогичного поведения нам необходимо написать свой drawable, что считаю нецелесообразным.

@temnik15
Copy link
Collaborator Author

temnik15 commented Feb 4, 2021

image

@temnik15 temnik15 linked an issue Feb 4, 2021 that may be closed by this pull request
3 tasks
@Shiplayer
Copy link
Contributor

image
Если так посмотреть, то как минимум цвет в туллбаре не совпадает с дизайном

@temnik15
Copy link
Collaborator Author

temnik15 commented Feb 4, 2021

Понял, цвет заголовка изменю

@temnik15
Copy link
Collaborator Author

temnik15 commented Feb 5, 2021

Поправил размер и цвет заголовка. Слева вариант из дизайна, справа из приложения (правый вариант чуть хуже качеством, так как это скриншот).
2

Ну и вот наложение, текст совпадает почти идеально.
1

@@ -21,6 +21,10 @@
<style name="MainToolbar">
<item name="android:background">@color/defaultBackgroundDarker</item>
<item name="colorControlNormal">@color/colorWhite</item>
<item name="titleTextColor">@color/colorWhite</item>
<item name="titleTextColor">@color/colorAppText</item>
<item name="android:fontFamily">@font/roboto_bold</item>
Copy link
Contributor

Choose a reason for hiding this comment

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

это можно было вынести в MainToolbar.TitileText, если не ошибаюсь, должно работать, тоже самое можно отнести к цвету.

Comment on lines 20 to 21
android:theme="@style/MainToolbar"
app:titleTextAppearance="@style/MainToolbar.TitleText" />
Copy link
Contributor

Choose a reason for hiding this comment

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

можем пока оставить это тут, но может это было бы лучше в стилях сразу определить для всех активити

Copy link
Contributor

Choose a reason for hiding this comment

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

вдруг нам где нибудь надо будет такой же туллбар использовать

@Shiplayer Shiplayer removed this from the MainActivity milestone Feb 12, 2021
…tle.

#64: corrected title color and size.

#64: moved attr toolbar to styles.
@temnik15 temnik15 force-pushed the feature/HAWK-64/create-toolbar-and-hamburger branch from b33bace to 78e5cff Compare February 12, 2021 21:46
@temnik15 temnik15 merged commit 056814d into main Feb 12, 2021
@temnik15 temnik15 deleted the feature/HAWK-64/create-toolbar-and-hamburger branch February 12, 2021 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Обновить туллбар
3 participants