Skip to content

Comments

Refactor/media details#34

Open
itsahmedsalah wants to merge 321 commits intoTheChance101:masterfrom
Copenhagen-Squad:refactor/media-details
Open

Refactor/media details#34
itsahmedsalah wants to merge 321 commits intoTheChance101:masterfrom
Copenhagen-Squad:refactor/media-details

Conversation

@itsahmedsalah
Copy link

This PR has the screen of details for both movie and tvshow
image
image

itsahmedsalah and others added 30 commits September 9, 2025 00:23
- Delete `item_actor_home.xml`, `item_actor_search.xml`, and `item_actor_see_all.xml`.
- Modify `item_actor.xml`:
    - Change root from `<layout>` to `<androidx.constraintlayout.widget.ConstraintLayout>`.
    - Add a `FrameLayout` (`@+id/image_container`) to hold actor image and placeholder icons.
    - Add `ImageView` for "no image" icon (`@+id/noImg_icon`) using `ic_no_actor_img.xml`.
    - Add `ImageView` for placeholder icon (`@+id/placeholder_icon`) using `ic_user`.
    - Make `ShapeableImageView` (`@+id/image_actor`) initially gone.
    - Remove data binding variables `listener` and `item`.
    - Update `TextView` (`@+id/text_actor_name`) to use `@string/actor_name`.
- Add new drawable `ic_no_actor_img.xml`.
- Add new string resources: `actor_placeholder`, `no_img`, `actor_name` in `values/strings.xml` and `values-night/strings.xml`.
- Update `ActorsAdapter.kt`:
    - Modify constructor to accept `layoutID` as a parameter.
- Update `ActorsFragment.kt`:
    - Pass `R.layout.item_actor` as `layoutID` to `ActorsAdapter`.
- Update `ActorSearchAdapter.kt`:
    - Change `layoutID` to `R.layout.item_actor`.
- Update `HomeAdapter.kt`:
    - Change `ActorAdapter` layout from `R.layout.item_actor_home` to `R.layout.item_actor`.
# Conflicts:
#	app/src/main/res/layout/item_actor.xml
#	app/src/main/res/layout/item_actor_home.xml
#	app/src/main/res/layout/item_actor_search.xml
Add new bottom navigation bar with dark theme support
Add custom cursor drawables and layout
Removes unnecessary comments and streamlines tab selection in `MainActivity`.
This layout defines a UI component for displaying informational messages with a title, caption, and two buttons.
MuhammedWael9991 and others added 30 commits September 13, 2025 23:36
# Conflicts:
#	app/src/main/java/com/karrar/movieapp/utilities/BindingAdapter.kt
#	app/src/main/res/layout/error_tv_show_details.xml
…ckages, and related UI states for improved clarity and consistency
…gation

Refactor list details to collection details, update profile navigation
…ails

# Conflicts:
#	app/src/main/res/values/strings.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants