Merge pull request #142 from MihaiCristianCondrea/codex/rename-xml-files-for-consistency

Align layout XML naming conventions

Author: MihaiCristianCondrea

app/src/main/java/com/d4rk/androidtutorials/java/ads/managers/NativeAdLoader.java

@@ -29,7 +29,7 @@ public class NativeAdLoader {
     private static final String TAG = "NativeAdLoader";
 
     public static void load(@NonNull Context context, @NonNull ViewGroup container) {
-        load(context, container, R.layout.large_home_banner_native_ad, null);
+        load(context, container, R.layout.ad_home_banner_large, null);
     }
 
     public static void load(@NonNull Context context, @NonNull ViewGroup container, @LayoutRes int layoutRes) {

app/src/main/java/com/d4rk/androidtutorials/java/ads/views/NativeAdBannerView.java

@@ -20,7 +20,7 @@
  */
 public class NativeAdBannerView extends FrameLayout {
 
-    private int layoutRes = R.layout.large_home_banner_native_ad;
+    private int layoutRes = R.layout.ad_home_banner_large;
 
     public NativeAdBannerView(@NonNull Context context) {
         super(context);
@@ -40,7 +40,7 @@ public NativeAdBannerView(@NonNull Context context, @Nullable AttributeSet attrs
     private void init(@NonNull Context context, @Nullable AttributeSet attrs, int defStyleAttr) {
         if (attrs != null) {
             TypedArray a = context.obtainStyledAttributes(attrs, R.styleable.NativeAdBannerView, defStyleAttr, 0);
-            layoutRes = a.getResourceId(R.styleable.NativeAdBannerView_nativeAdLayout, R.layout.large_home_banner_native_ad);
+            layoutRes = a.getResourceId(R.styleable.NativeAdBannerView_nativeAdLayout, R.layout.ad_home_banner_large);
             a.recycle();
         }
     }

app/src/main/java/com/d4rk/androidtutorials/java/ui/components/navigation/BottomSheetMenuFragment.java

@@ -11,19 +11,19 @@
 
 import com.d4rk.androidtutorials.java.BuildConfig;
 import com.d4rk.androidtutorials.java.R;
-import com.d4rk.androidtutorials.java.databinding.BottomSheetMenuBinding;
+import com.d4rk.androidtutorials.java.databinding.DialogBottomSheetMenuBinding;
 import com.d4rk.androidtutorials.java.ui.screens.settings.SettingsActivity;
 import com.google.android.material.bottomsheet.BottomSheetDialogFragment;
 
 public class BottomSheetMenuFragment extends BottomSheetDialogFragment {
 
-    private BottomSheetMenuBinding binding;
+    private DialogBottomSheetMenuBinding binding;
 
     @Override
     public View onCreateView(@NonNull LayoutInflater inflater,
                              ViewGroup container,
                              Bundle savedInstanceState) {
-        binding = BottomSheetMenuBinding.inflate(inflater, container, false);
+        binding = DialogBottomSheetMenuBinding.inflate(inflater, container, false);
 
         binding.menuSettings.setOnClickListener(v -> {
             Intent intent = new Intent(requireContext(), SettingsActivity.class);

app/src/main/java/com/d4rk/androidtutorials/java/ui/screens/android/AndroidStudioFragment.java

@@ -341,15 +341,15 @@ public RecyclerView.ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int
                 adView.setLayoutParams(new RecyclerView.LayoutParams(
                         ViewGroup.LayoutParams.MATCH_PARENT,
                         ViewGroup.LayoutParams.WRAP_CONTENT));
-                adView.setNativeAdLayout(R.layout.android_studio_list_native_ad);
+                adView.setNativeAdLayout(R.layout.ad_android_studio_list);
                 return new AdHolder(adView);
             } else if (viewType == TYPE_CATEGORY) {
                 View view = LayoutInflater.from(parent.getContext())
-                        .inflate(R.layout.android_studio_category_item, parent, false);
+                        .inflate(R.layout.item_android_studio_category, parent, false);
                 return new CategoryHolder(view);
             } else {
                 View view = LayoutInflater.from(parent.getContext())
-                        .inflate(R.layout.android_studio_lesson_item, parent, false);
+                        .inflate(R.layout.item_android_studio_lesson, parent, false);
                 return new LessonHolder(view);
             }
         }

app/src/main/java/com/d4rk/androidtutorials/java/ui/screens/home/HomeFragment.java

@@ -55,8 +55,8 @@ public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceStat
             java.util.List<PromotedApp> apps = state.promotedApps();
             for (int i = 0; i < apps.size(); i++) {
                 PromotedApp app = apps.get(i);
-                com.d4rk.androidtutorials.java.databinding.PromotedAppItemBinding itemBinding =
-                        com.d4rk.androidtutorials.java.databinding.PromotedAppItemBinding.inflate(inflater, promotedContainer, false);
+                com.d4rk.androidtutorials.java.databinding.ItemPromotedAppBinding itemBinding =
+                        com.d4rk.androidtutorials.java.databinding.ItemPromotedAppBinding.inflate(inflater, promotedContainer, false);
                 loadImage(app.iconUrl(), itemBinding.appIcon);
                 itemBinding.appName.setText(app.name());
                 itemBinding.appDescription.setVisibility(android.view.View.GONE);

app/src/main/res/layout/activity_main.xml

@@ -57,7 +57,7 @@
             android:id="@+id/ad_view"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            app:nativeAdLayout="@layout/bottom_app_bar_native_ad" />
+            app:nativeAdLayout="@layout/ad_bottom_app_bar" />
     </FrameLayout>
 
     <com.google.android.material.bottomnavigation.BottomNavigationView

app/src/main/res/layout/activity_support.xml

@@ -126,7 +126,7 @@
                 android:layout_height="wrap_content"
                 android:layout_marginTop="16dp"
                 android:layout_marginBottom="16dp"
-                app:nativeAdLayout="@layout/large_home_banner_native_ad" />
+                app:nativeAdLayout="@layout/ad_home_banner_large" />
         </androidx.appcompat.widget.LinearLayoutCompat>
     </me.zhanghai.android.fastscroll.FastScrollScrollView>
 </androidx.constraintlayout.widget.ConstraintLayout>

app/src/main/res/layout/about_native_ad.xml renamed to app/src/main/res/layout/ad_about.xml

@@ -9,20 +9,21 @@
         android:layout_height="wrap_content"
         android:orientation="vertical"
         android:background="?attr/colorSurfaceContainer"
-        android:paddingStart="16dp"
-        android:paddingEnd="16dp"
-        android:paddingTop="12dp"
-        android:paddingBottom="12dp">
+        android:paddingHorizontal="16dp">
 
-        <include layout="@layout/ad_attribution" />
+        <include
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="4dp"
+            layout="@layout/ad_attribution" />
 
         <LinearLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:gravity="center_vertical"
             android:orientation="horizontal">
 
-            <ImageView
+            <androidx.appcompat.widget.AppCompatImageView
                 android:id="@+id/ad_app_icon"
                 android:layout_width="32dp"
                 android:layout_height="32dp"