-
Notifications
You must be signed in to change notification settings - Fork 565
Fix #1954 : Optimise profile chooser fragment #1997
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 #1954 : Optimise profile chooser fragment #1997
Conversation
@rt4914 PTAL |
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.
Nice implementation.
Some nit changes suggested.
@peculiaruc Also, please mark the checklist in PR description. You can do that by placing |
Will correct those thanks
…On Tue, 13 Oct 2020, 7:54 AM Rajat Talesra, ***@***.***> wrote:
Assigned #1997 <#1997> to
@peculiaruc <https://github.com/peculiaruc>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#1997 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIOVAV2DXJYKFK363Z3IORDSKP2RZANCNFSM4SM6AA7Q>
.
|
@rt4914 PTAL |
Done @rajat Talesra <[email protected]> , PTAL
On Tue, 13 Oct 2020 at 08:51, Peculiar Umeh <[email protected]>
wrote:
… Will correct those thanks
On Tue, 13 Oct 2020, 7:54 AM Rajat Talesra, ***@***.***>
wrote:
> Assigned #1997 <#1997> to
> @peculiaruc <https://github.com/peculiaruc>.
>
> —
> You are receiving this because you were assigned.
> Reply to this email directly, view it on GitHub
> <#1997 (comment)>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AIOVAV2DXJYKFK363Z3IORDSKP2RZANCNFSM4SM6AA7Q>
> .
>
|
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.
* Optimise code in profile_chooser_fragment xml * {{Fixes oppia#1954: Optimise code in profile chooser fragment xml}} * {{Fixes oppia#1997:Optimise code in profile chooser fragment xml}} Co-authored-by: pecpaker <[email protected]>
* Optimise code in profile_chooser_fragment xml * {{Fixes oppia#1954: Optimise code in profile chooser fragment xml}} * {{Fixes oppia#1997:Optimise code in profile chooser fragment xml}} Co-authored-by: pecpaker <[email protected]>
Explanation
"Fixes #1954:" Optimized profile chooser fragment.xml by adding style to TextViews.'
Checklist