-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
Closed
Labels
GUIIssue is related to the graphical user interfaceIssue is related to the graphical user interfacebugIssue is related to a bugIssue is related to a bug
Description
Model: Samsung N9500
Screen display: Medium scale, medium font
Actually, I think the list_stream_item height is a little low, so I just change it to 110dp, just look fine.
My suggestion is:
- Change the layout to a better one (which I'm not sure, maybe constraintlayout?)
- Add an option of having a large item view which could bypass this issue (Youtube client has a way bigger item)
FYI:
Before/after
Before:
<!-- Elements Size -->
<!-- 16 / 9 ratio-->
<dimen name="video_item_search_thumbnail_image_width">124dp</dimen>
<dimen name="video_item_search_thumbnail_image_height">70dp</dimen>
<!-- Calculated: 2*video_item_search_padding + video_item_search_thumbnail_image_height -->
<dimen name="video_item_search_height">94dp</dimen>
After:
<!-- Elements Size -->
<!-- 16 / 9 ratio-->
<dimen name="video_item_search_thumbnail_image_width">150dp</dimen>
<dimen name="video_item_search_thumbnail_image_height">86dp</dimen>
<!-- Calculated: 2*video_item_search_padding + video_item_search_thumbnail_image_height -->
<dimen name="video_item_search_height">110dp</dimen>
tbm
Metadata
Metadata
Assignees
Labels
GUIIssue is related to the graphical user interfaceIssue is related to the graphical user interfacebugIssue is related to a bugIssue is related to a bug