-
Notifications
You must be signed in to change notification settings - Fork 503
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
Merge topic_practice_subtopic.xml into single xml file #3586
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
@rt4914 I would like to work on this issue, kindly assign it to me. |
@kritigupta45 |
cc: @Arjupta |
6 tasks
CLAM Team (deprecated -- please use new board)
automation
moved this from Backlog (things to organize into workstreams)
to Closed
Oct 3, 2021
rt4914
pushed a commit
that referenced
this issue
Oct 3, 2021
…3835) * merge topic_practice_subtopic xml files & create variables in dimens.xml file * revert unnecessary changes * modify files for mob-landscape view * changes dimens.xml * revert .idea file * modify xml file for tablet mode
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.
Currently there are 4 versions of
topic_practice_subtopic.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
dimens.xml
file and use it accordingly.Note: In PR, make sure you add before and after screenshot of mobile-portrait, mobile-landscape, tablet-portrait and tablet-landscape for comparison and make sure that there is not difference between before and after UI.
The text was updated successfully, but these errors were encountered: