Skip to content

Commit

Permalink
Fix #3584: Merge 4 versions of topic_practice_fragment.xml into a sin…
Browse files Browse the repository at this point in the history
…gle xml file (#4022)

* merge files & changes dimen files accordingly

* resolve merge conflict

* modified dimens file

* nit fixes

* change tablet files

* sw600dp file changes
  • Loading branch information
kritigupta45 committed Jan 18, 2022
1 parent 3d876dc commit 034b55f
Show file tree
Hide file tree
Showing 8 changed files with 23 additions and 56 deletions.
26 changes: 0 additions & 26 deletions app/src/main/res/layout-land/topic_practice_fragment.xml

This file was deleted.

26 changes: 0 additions & 26 deletions app/src/main/res/layout-sw600dp-port/topic_practice_fragment.xml

This file was deleted.

5 changes: 4 additions & 1 deletion app/src/main/res/layout/topic_practice_fragment.xml
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,12 @@
android:id="@+id/topic_practice_skill_list"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:background="@color/oppia_background"
android:clipToPadding="false"
android:overScrollMode="never"
android:paddingBottom="172dp"
android:paddingStart="@dimen/topic_practice_fragment_padding_start"
android:paddingEnd="@dimen/topic_practice_fragment_padding_end"
android:paddingBottom="@dimen/topic_practice_fragment_padding_bottom"
android:scrollbars="none"
app:data="@{viewModel.topicPracticeSkillLiveData}"
app:layout_constraintBottom_toBottomOf="parent"
Expand Down
7 changes: 6 additions & 1 deletion app/src/main/res/values-land/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -390,10 +390,15 @@
<dimen name="section_title_text_view_layout_margin_top">28dp</dimen>
<dimen name="section_title_text_view_layout_margin_end">72dp</dimen>

<!-- TopicPracticeFragment -->
<dimen name="topic_practice_fragment_padding_bottom">0dp</dimen>
<dimen name="topic_practice_fragment_padding_start">48dp</dimen>
<dimen name="topic_practice_fragment_padding_end">48dp</dimen>

<!-- PromotedStoryList-->
<dimen name="promoted_story_list_layout_margin_end">72dp</dimen>
<dimen name="promoted_story_list_layout_margin_start">72dp</dimen>

<!-- AudioLanguageFragment -->
<dimen name="audio_language_recycler_view_padding_bottom">96dp</dimen>

Expand Down
5 changes: 4 additions & 1 deletion app/src/main/res/values-sw600dp-land/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -396,10 +396,13 @@
<dimen name="section_title_text_view_layout_margin_top">32dp</dimen>
<dimen name="section_title_text_view_layout_margin_end">96dp</dimen>

<!-- TopicPracticeFragment -->
<dimen name="topic_practice_fragment_padding_bottom">112dp</dimen>

<!-- PromotedStoryList-->
<dimen name="promoted_story_list_layout_margin_end">96dp</dimen>
<dimen name="promoted_story_list_layout_margin_start">96dp</dimen>

<!-- AudioLanguageFragment -->
<dimen name="audio_language_recycler_view_padding_bottom">132dp</dimen>

Expand Down
5 changes: 4 additions & 1 deletion app/src/main/res/values-sw600dp-port/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -392,12 +392,15 @@
<dimen name="solution_summary_margin_bottom_linear_layout">32dp</dimen>
<dimen name="solution_summary_margin_top_solution_summary_label">16dp</dimen>

<!-- SectionTitleFragment -->
<!-- SectionTitleFragment -->
<dimen name="section_title_divider_view_layout_margin_top">48dp</dimen>
<dimen name="section_title_text_view_layout_margin_start">120dp</dimen>
<dimen name="section_title_text_view_layout_margin_top">32dp</dimen>
<dimen name="section_title_text_view_layout_margin_end">120dp</dimen>

<!-- TopicPracticeFragment -->
<dimen name="topic_practice_fragment_padding_bottom">112dp</dimen>

<!-- PromotedStoryList-->
<dimen name="promoted_story_list_layout_margin_end">120dp</dimen>
<dimen name="promoted_story_list_layout_margin_start">120dp</dimen>
Expand Down
5 changes: 5 additions & 0 deletions app/src/main/res/values/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -586,6 +586,11 @@
<dimen name="section_title_text_view_layout_margin_top">28dp</dimen>
<dimen name="section_title_text_view_layout_margin_end">32dp</dimen>

<!-- TopicPracticeFragment -->
<dimen name="topic_practice_fragment_padding_bottom">172dp</dimen>
<dimen name="topic_practice_fragment_padding_start">0dp</dimen>
<dimen name="topic_practice_fragment_padding_end">0dp</dimen>

<!-- PromotedStoryList-->
<dimen name="promoted_story_list_layout_margin_end">28dp</dimen>
<dimen name="promoted_story_list_layout_margin_start">28dp</dimen>
Expand Down

0 comments on commit 034b55f

Please sign in to comment.