-
Notifications
You must be signed in to change notification settings - Fork 509
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
Fix part of #41: Hifi Story List #615
Conversation
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.
@Luffy18346 PTAL
@Luffy18346 please add the final screenshot of your work in the description of the PR. |
@Luffy18346 One more change, the tick mark used in develop code is different than the tick mark in mocks. Update the tick mark in |
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.
Just one more change, remove the unused ic_check_primary
icon.
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.
Please remove auto-changed code.
.idea/misc.xml
Outdated
@@ -6,4 +6,4 @@ | |||
<component name="ProjectType"> | |||
<option name="id" value="Android" /> | |||
</component> | |||
</project> | |||
</project> |
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.
Revert these file changes.
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.
Changes reverted in this file.
<option name="myName" value="Project Default" /> | ||
<inspection_tool class="IncorrectScope" enabled="false" level="ERROR" enabled_by_default="false" /> | ||
</profile> | ||
</component> |
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.
Revert these file changes.
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.
This file is removed.
.idea/codeStyles/Project.xml
Outdated
@@ -157,4 +162,4 @@ | |||
</indentOptions> | |||
</codeStyleSettings> | |||
</code_scheme> | |||
</component> | |||
</component> |
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.
Revert these file changes.
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.
Changes reverted in this file.
@Luffy18346 always make sure that before your PR gets closed or merged you have replied to all the conversations and resolved all conversations. |
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.
LGTM, thanks.
Explanation
Fix part of #41: Changes UI of Story header and story list item in StoryList
Checklist