akazadd / NimbleTechTest

This repository contains the iOS technical test project for Nimble - a demonstration of skills in iOS development for the technical evaluation process.
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 #22

Open minhnimble opened 1 year ago

minhnimble commented 1 year 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:

Example screenshot:

Expect

akazadd commented 1 year ago

I am currently facing two issues that require your assistance:

High-Resolution Images: I am experiencing difficulty obtaining high-resolution images from the API response. According to the documentation, appending "l" to the image URL is intended to provide a higher resolution. However, this method does not seem to be working as expected. I kindly request your guidance on how to retrieve high-resolution images correctly. If there are specific guidelines or alternative methods that need to be followed, please provide the necessary details.

Required Font (Neuzeit S LT Std): Additionally, I am in need of the Neuzeit S LT Std font for use in our project. Could you please provide a download link or any relevant resources for acquiring this font?

minhnimble commented 1 year ago

...appending "l" to the image URL is intended to provide a higher resolution. However, this method does not seem to be working as expected.

Can you explain a bit more specific on what you have tried so far, and what was the error back then? As a small reminder, the instruction should be relatively simple that append ā€œlā€ to the image URL obtained in the API response.

Additionally, I am in need of the Neuzeit S LT Std font for use in our project. Could you please provide a download link or any relevant resources for acquiring this font?

As a matter of fact, you can use an alternative font for this, as long as it belongs to Neuzeit S LT Std font family (e.g., Neuzeit S LT Std Book) šŸ‘Œ This should be a challenge for you to figure it out šŸ™‚

akazadd commented 1 year ago

Please check this pull request for this issue https://github.com/akazadd/NimbleTechTest/pull/30

minhnimble commented 1 year ago

Please check this pull request for this issue https://github.com/akazadd/NimbleTechTest/pull/30

I am getting quite confused now, did you check the image sizes that our server returns per URL, one with the l postfix and one that doesn't? For example, from one of the response payloads:

{
    attributes =     {
        "active_at" = "2017-01-18T09:13:00.000Z";
        "cover_image_url" = "https://dhdbhh0jsld0o.cloudfront.net/m/59e9e7327354006c1ebc_";
        "created_at" = "2017-01-18T09:13:55.081Z";
        description = "We'd love to hear from you!";
        "inactive_at" = "<null>";
        "is_active" = 1;
        "survey_type" = Wellness;
        "thank_email_above_threshold" = "<null>";
        "thank_email_below_threshold" = "<null>";
        title = "Health Land Spa";
    };
    id = 5d2538f53ca50536292c;
    type = "survey_simple";
}

=> https://dhdbhh0jsld0o.cloudfront.net/m/59e9e7327354006c1ebc_ vs. https://dhdbhh0jsld0o.cloudfront.net/m/59e9e7327354006c1ebc_l are they showing the same image on your side?

akazadd commented 1 year ago

I have verified the image sizes from the server URLs, both with and without the 'l' postfix. It appears that they represent the same image but with different sizes.