-
Notifications
You must be signed in to change notification settings - Fork 3
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
brand new home screen #42
Conversation
shalva97
commented
May 7, 2022
@@ -1,6 +1,6 @@ | |||
<resources> | |||
<string name="app_name">Testomania</string> | |||
<string name="general_skills_tests">უნარების ტესტები</string> | |||
<string name="general_skills_tests">უნარების მათემატიკური ტესტები</string> | |||
<string name="driving_license_tests">მარვის მოწმობის ტესტები</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: typo here, should be: მართვის instead of მარვის
I like that we are using icons alongside the text very much, but I thought we were using a more squarish approach, more like this: |
არ გამოდიოდა ლამაზად, იმ სურათში ყველაფერს ერთი სიტყვა აწერია, ჩვენ შემთხვევაში მინიმუმ 2 სიტყვა არის |
# Conflicts: # testomania/src/main/java/com/earth/testomania/presentation/home/HomeScreen.kt