Fix part of #23: Topic Tab Icons updated

[rt4914]

Explanation

Topic Tabs updated icons:

Checklist

• The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes. (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
• The PR explanation includes the words "Fixes #bugnum: ..." (or "Fixes part of #bugnum" if the PR only partially fixes an issue).
• The PR follows the style guide.
• The PR does not contain any unnecessary auto-generated code from Android Studio.
• The PR is made from a branch that's not called "develop".
• The PR is made from a branch that is up-to-date with "develop".
• The PR's branch is based on "develop" and not on any other branch.
• The PR is assigned to an appropriate reviewer in both the Assignees and the Reviewers sections.

[rt4914]

@mschanteltc These icons are way better, but I still think that topic-train icon is comparatively smaller than others, can we fix that?

Thanks.

[mschanteltc]

@rt4914 I updated the train icon so that the height and width are equal. All the other icons LGTM.

[rt4914]

@rt4914 I updated the train icon so that the height and width are equal. All the other icons LGTM.

Yeah, the updated train icon looks perfect. Check description for screenshot.

[mschanteltc]

It seems that the stroke width of the train icon makes the set of icons appear inconsistent. I updated the train icon again with a different brain image and omitted the weight. Can you update the mock with this new icon please?

[rt4914]

It seems that the stroke width of the train icon makes the set of icons appear inconsistent. I updated the train icon again with a different brain image and omitted the weight. Can you update the mock with this new icon please?

Check this @mschanteltc

[mschanteltc]

Looks good! One last thing is to make sure the sizing of icons fits the Material Guidelines. I updated the Review Icon to follow these standards, so please update this on mocks.

Overview, Play, and Train icons (circle icons):

• Height: 20 dp

• Width: 20 dp

Review icon (vertical rectangle icon):

• Height: 20 dp

• Width: 16 dp

[nikitamarysolomanpvt]

Design looks good to me please address @mschanteltc's suggestions and assign me back.

[rt4914]

Looks good! One last thing is to make sure the sizing of icons fits the Material Guidelines. I updated the Review Icon to follow these standards, so please update this on mocks.

Overview, Play, and Train icons (circle icons):

• Height: 20 dp
• Width: 20 dp

Review icon (vertical rectangle icon):

• Height: 20 dp
• Width: 16 dp

Used all latest icons from the drive.

Result:

[nikitamarysolomanpvt]

LGTM

[mschanteltc]

LGTM!