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

[Tabs] replace text by icons for kids. #14

Merged
merged 1 commit into from Jul 30, 2016
Merged

Conversation

tbarthel-fr
Copy link
Member

Fab button as been changed to a '+' icon for not using
same icons at different place. In addition, it could
help to understand the action behind the FAB.

for kids :
icons

default :

text

@vbarthel-fr
Copy link
Member

lgtm 👍

@vbarthel-fr vbarthel-fr removed their assignment Jul 23, 2016
@tbarthel-fr
Copy link
Member Author

adult version : keep icons instead of + icons.

@jo-elimu
Copy link
Member

@tbarthel-fr Actually, kid's who have not learned literacy/numeracy yet do not know what '+' means, so icons should be of shapes/figures rather than characters.

@vbarthel-fr
Copy link
Member

Make sense, thanks for the clarification =)

@tbarthel-fr Do you think we could re-use the "drop" allusion you used in #15. One drop for picking a color, 3 "merging drops" for creating a palette?

@tbarthel-fr
Copy link
Member Author

@jogrimst thanks for this precious feedback! We are going to rework this icon asap (=

@tbarthel-fr
Copy link
Member Author

@jogrimst Here is another attempt to replace text used as tab title by icons.

This time, original FAB icons as been kept since their represent the best which action is behind. And we simply use shapes to replace tab title.

device-2016-07-25-122027

device-2016-07-25-122056

device-2016-07-25-122254

device-2016-07-25-122642

@jo-elimu
Copy link
Member

@tbarthel-fr I think the circle vs square is a good solution. Maybe the square could be replaced with the "view_column" icon also? https://github.com/google/material-design-icons/blob/master/action/2x_web/ic_view_column_black_48dp.png

@jo-elimu
Copy link
Member

@tbarthel-fr I just tested the app on a Google Pixel C device. And it was difficult to see the white color, both in the "selected colors" tab, and in the activity for creating a new palette. Maybe the white color should behave slightly different? Or put some mild shadow behind each circle?

screenshot_20160727-131833

@tbarthel-fr
Copy link
Member Author

Hi @jogrimst (=

Ok for the shape! Here is a screenshot with the column icon you suggested. I let you choose between square and column before submitting it for review (:

device-2016-07-28-082241

Regarding the white color, we definitely have to find a trick for every color close enough to the background one (x we will come back to you with a proposition as soon as we would have one (: (probably some kinds of shadow). I'm going to create an issue to track this down.

Regarding the tablette support, we are going to submit you some small changes which must improve the user experience on 9' and 10' devices. Basically, most of this small changes are going to be dimen tweaks. A new PR will be created soon.

Best regards,

Thomas.

@jo-elimu
Copy link
Member

Ok for the shape! Here is a screenshot with the column icon you suggested. I let you choose between square and column before submitting it for review (:

@tbarthel-fr Thanks. I think the column variant looks a bit better than the square for the icon.

@tbarthel-fr
Copy link
Member Author

@jogrimst Perfect (= I mark this feature as validated and wait for @vbarthel-fr review!

@vbarthel-fr
Copy link
Member

@tbarthel-fr the solution adopted does not seem to use the ic_add drawable: the resource is unused.

Otherwise, lgtm 👍

picker and palette icons has been kept for the
fab since it's represent well the action behind.
Color and Palette words of the tabs have been replaced
by icons which represent what kids are going to find
in the list : rounds for colors and squares for palettes.
@tbarthel-fr tbarthel-fr merged commit ff17150 into kid-matters Jul 30, 2016
@tbarthel-fr tbarthel-fr deleted the tb/tab-icons branch July 30, 2016 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants