[Bug] The survey item content on the Home screen is difficult to see and doesn't match with the design

hsu-hsu / nimble-survey-app

This is an application that allows users to browse a list of surveys.

0 stars 0 forks source link

[Bug] The survey item content on the Home screen is difficult to see and doesn't match with the design #17

Closed minhnimble closed 8 months ago

minhnimble commented 8 months ago

Issue

The application needs to show the survey list after loading all the data with a valid logged-in session-like design. Currently, there are multiple UI problems with a survey item on the Home screen:

The display image for each survey item is very blurry.
The Today and date texts are not shown as per the design.
The default avatar icon doesn't show as per the design.
The left indentation between the survey title, description and the page indicator isn't properly aligned per design.
The page indicator UI doesn't match with the design.

Example screenshot:

Expect

The survey item should look clearer and identical to the design:

hsu-hsu commented 8 months ago

In this pull request, I add today text, today date and avatar image in home screen as design. I also fixed pager indicator design and align the left margin of textview and pager indicator as design.

For the display blurry bug, I think there is a problem between the aspect ratios of the image we received from api and the one we want to display. For the full image design, I used centercrop in scale type. That is the reason image is blurry. If we use fitcenter in scale type, the image will be more clear but it won't be full screen. I only set scale type to centercrop in this pull request because if we can get the desired aspect ratio, it will be the best option for this kind of display.

pull request -> #25

minhnimble commented 8 months ago

For the display blurry bug, I think there is a problem between the aspect ratios of the image we received from api and the one we want to display.

Hi @hsu-hsu, just sharing here again in case you miss, the API can support high res images with instructions from Postman. 😁

hsu-hsu commented 8 months ago

Hello Mike,

Thank you for your information. I oversee that one while reading API documentation😅. I will replace image with high resolution image in the home screen.

hsu-hsu commented 8 months ago

In this pull request, I fix image blurry bug in home screen. pull request -> #28