Fix part of #24: Concept card for review [Blocked on #331]

[rt4914]

Explanation

Everything in this PR is done, except for changing the close icon on top-left same as mocks.

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.

[BenHenning]

Thanks @rt4914. Generally LGTM, but please add a test to verify that the html parsing is happening. I believe we can verify this by retrieving the CharSequence from the TextView, casting it to a SpannableString, and then verify there are expected Spans present in the string.

[jamesxu0]

@rt4914 I would like to merge PR #331 before this one, which creates a recyclerview with the heading, explanation, and worked examples and also test cases with rich text checking.

[rt4914]

@rt4914 I would like to merge PR #331 before this one, which creates a recyclerview with the heading, explanation, and worked examples and also test cases with rich text checking.

@jamesxu0 Acknowledge. Will wait for your merge and also some test cases needs change in ID name to fix them, will do that too.

[veena14cs]

LGTM

[jamesxu0]

@rt4914 PR #331 has been merged into develop. I used a lot of the stylings that were in this PR for concept_card_fragment.xml, so this PR is probably not needed.

[rt4914]

@rt4914 PR #331 has been merged into develop. I used a lot of the stylings that were in this PR for concept_card_fragment.xml, so this PR is probably not needed.

Thanks a lot @jamesxu0 I will close this PR, and check everything on develop and close the hifi issue accordingly.