search

RachmanForniandi / Dicoding-Events-Tracker

Submission Class android Fundamental
0 stars 0 forks source link

Implementation NestedScrollView and GridLayoutManger #4

Open algokelvin-373 opened 2 weeks ago

algokelvin-373 commented 2 weeks ago

Screenshot 2024-08-31 143809

I look this page. That isn't good especially looked by UX. So, I give you 2 points recommendation:

  1. Implement with GridLayoutManager I very recommended to use this. That's good looking for UI/UX. Don't forget to setting width and height image. That's bigger. Setting like that width: 100dp and height: 100dp (example).

  2. Implement with NestedScrollView If you want to scroll which that page has two recyclerview, you should be implement NestedScrollView. That's better step to make Home Page can be scrolling.

So, this is my note for you. I will check your code. Thanks bro @RachmanForniandi

RachmanForniandi commented 1 week ago

sorry for late reply. From your suggestion, here is my reply per point:

  1. I think I am gonna keep using layoutmanager for history event display data on home page tab because if using gridlayoutmanager, it would be too small & not fully exposed especially for name of speakers. Even using scaletype center crop or fit XY can't help it. Also I can't determine the size of textview of title event.
  2. From this point, I am agree to add nestedscrollview on root view of home page event. It performed better than regular scrollview.