Skip to content

Commit

Permalink
Fix #3359: Convert welcome.xml into single layout file pull request f…
Browse files Browse the repository at this point in the history
…or issue (#3368)

* Fix ##3359 Convert welcome.xml into single layout file

* Created new dimens value for home welcome outer padding

* reverting back to previous commits in .idea/misc.xml .idea/runConfigurations.xml build.gradle

* changing files .idea/misc.xml .idea/runConfigurations.xml build.gradle to as per develop branch

* Resolving some commit issues in .idea/misc.xml .idea/runConfigurations.xml build.gradle

* Revert back to prevoius changes in .idea/misc.xml
  • Loading branch information
TheSwarnim authored Jun 29, 2021
1 parent dcd38c1 commit 8dcd8de
Show file tree
Hide file tree
Showing 8 changed files with 11 additions and 183 deletions.
60 changes: 0 additions & 60 deletions app/src/main/res/layout-land/welcome.xml

This file was deleted.

60 changes: 0 additions & 60 deletions app/src/main/res/layout-sw600dp-land/welcome.xml

This file was deleted.

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

This file was deleted.

6 changes: 3 additions & 3 deletions app/src/main/res/layout/welcome.xml
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
<androidx.constraintlayout.widget.ConstraintLayout
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:paddingStart="28dp"
android:paddingEnd="28dp">
android:paddingStart="@dimen/home_welcome_outer_padding"
android:paddingEnd="@dimen/home_welcome_outer_padding">

<com.google.android.flexbox.FlexboxLayout
android:id="@+id/welcome_message_container"
Expand All @@ -31,7 +31,7 @@
android:id="@+id/welcome_text_view"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginEnd="4dp"
android:layout_marginEnd="@dimen/home_welcome_text_view_margin_end"
android:fontFamily="sans-serif"
android:text="@{viewModel.greeting}"
android:textColor="@color/oppiaPrimaryText"
Expand Down
2 changes: 2 additions & 0 deletions app/src/main/res/values-land/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
<dimen name="home_padding_end">64dp</dimen>
<dimen name="home_padding_start">64dp</dimen>
<dimen name="home_outer_margin">72dp</dimen>
<dimen name="home_welcome_outer_padding">72dp</dimen>
<dimen name="home_welcome_text_view_margin_end">8dp</dimen>
<dimen name="home_inner_margin">36dp</dimen>
<dimen name="recently_played_margin_max">72dp</dimen>
<dimen name="recently_played_margin_min">36dp</dimen>
Expand Down
2 changes: 2 additions & 0 deletions app/src/main/res/values-sw600dp-land/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
<dimen name="home_padding_end">64dp</dimen>
<dimen name="home_padding_start">96dp</dimen>
<dimen name="home_outer_margin">96dp</dimen>
<dimen name="home_welcome_outer_padding">96dp</dimen>
<dimen name="home_welcome_text_view_margin_end">8dp</dimen>
<dimen name="home_inner_margin">64dp</dimen>
<dimen name="recently_played_margin_max">96dp</dimen>
<dimen name="recently_played_margin_min">64dp</dimen>
Expand Down
2 changes: 2 additions & 0 deletions app/src/main/res/values-sw600dp-port/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
<dimen name="home_padding_end">96dp</dimen>
<dimen name="home_padding_start">120dp</dimen>
<dimen name="home_outer_margin">120dp</dimen>
<dimen name="home_welcome_outer_padding">120dp</dimen>
<dimen name="home_welcome_text_view_margin_end">8dp</dimen>
<dimen name="home_inner_margin">60dp</dimen>
<dimen name="recently_played_margin_max">120dp</dimen>
<dimen name="recently_played_margin_min">60dp</dimen>
Expand Down
2 changes: 2 additions & 0 deletions app/src/main/res/values/dimens.xml
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,8 @@
<dimen name="home_padding_end">20dp</dimen>
<dimen name="home_padding_start">20dp</dimen>
<dimen name="home_outer_margin">28dp</dimen>
<dimen name="home_welcome_outer_padding">28dp</dimen>
<dimen name="home_welcome_text_view_margin_end">4dp</dimen>
<dimen name="home_inner_margin">8dp</dimen>
<dimen name="home_tablet_land_outer_margin">96dp</dimen>
<dimen name="home_tablet_port_outer_margin">120dp</dimen>
Expand Down

0 comments on commit 8dcd8de

Please sign in to comment.