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

Convert faq_single_activity into single layout file #3474

Closed
TheSwarnim opened this issue Jul 13, 2021 · 3 comments · Fixed by #3526
Closed

Convert faq_single_activity into single layout file #3474

TheSwarnim opened this issue Jul 13, 2021 · 3 comments · Fixed by #3526
Assignees
Labels
good first issue This item is good for new contributors to make their pull request. Priority: Nice-to-have This work item is nice to have for its milestone. Z-ibt Temporary label for Ben to keep track of issues he's triaged.

Comments

@TheSwarnim
Copy link
Contributor

Currently there are 3 versions of faq_single_activity .xml file merge it into single xml file.

We can use https://text-compare.com/ to compare two versions of this file and for all the differences we can create variables in the dimens.xml file and use it accordingly.

Note: In PR, make sure you add before and after screenshots of mobile-portrait, mobile-landscape, tablet-portrait, and tablet-landscape for comparison and make sure that there is no difference between before and after UI.

@anandwana001 anandwana001 added good first issue This item is good for new contributors to make their pull request. Type: Improvement Priority: Nice-to-have This work item is nice to have for its milestone. labels Jul 14, 2021
@rt4914 rt4914 added this to Needs Triage in CLAM Team (deprecated -- please use new board) via automation Jul 15, 2021
@rt4914 rt4914 modified the milestones: Alpha v2, Backlog Jul 15, 2021
@rt4914 rt4914 moved this from Needs Triage to Backlog (things to organize into workstreams) in CLAM Team (deprecated -- please use new board) Jul 15, 2021
@rishidyno
Copy link
Contributor

Sir, i would like to work on this issue.

@anandwana001
Copy link
Contributor

Hi @rishidyno
Any update on this issue? Any difficulties you are facing while working on this issue?

@rishidyno
Copy link
Contributor

rishidyno commented Jul 24, 2021

@anandwana001 I have added a pull request for the same with some details.

#3518

CLAM Team (deprecated -- please use new board) automation moved this from Backlog (things to organize into workstreams) to Closed Aug 6, 2021
rt4914 pushed a commit that referenced this issue Aug 6, 2021
* in progress

* in progress

* in progress

* in progress

* merged faq_single_activity.xml into one file

* merged faq_single_activity.xml into one file

* merged faq_single_activity.xml into one file

* added EOL

* Added EOL to booleans.xml sw600-dp

* deleted 3 versions of topic_practice_footer_view.xml and merged it into one updated

* updated project

* updating project

* updating project

* Renamed to faq_single_activity_scroll_padding_Bottom to faq_single_activity_scroll_padding_bottom

* Renamed to faq_single_activity_scroll_padding_Bottom to faq_single_activity_scroll_padding_bottom

* Renamed to faq_single_activity_scroll_padding_Bottom to faq_single_activity_scroll_padding_bottom

* Renamed to faq_single_activity_scroll_padding_Bottom to faq_single_activity_scroll_padding_bottom

* Delete deploymentTargetDropDown.xml

* deleted boolenas.xml sw600dp-land 

Now we are having 2 versions of faq_single_activity we don't need to mention fillViewport attribute so i have deleted this file

* deleted unnecessary fillviewPort attribute from activity

* Deleted 2 unnecessary attributes in default layout

* Delete deploymentTargetDropDown.xml
@BenHenning BenHenning added the Z-ibt Temporary label for Ben to keep track of issues he's triaged. label Sep 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue This item is good for new contributors to make their pull request. Priority: Nice-to-have This work item is nice to have for its milestone. Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Projects
No open projects
5 participants