@vanzasetia
Posted
Hi, Faruk! 👋
I recommend reading this CSS Tricks article to make the text color adjust based on the background color. After, you know how it works, I recommend removing the "hack" and then using the better approach to achieve it.
I suggest using CSS to uppercase the text. The uppercased word in the HTML might be spelled by the screen reader (spelled letter by letter).
On 870px width, there is an issue where the image-hero
is touching with the next section. So, I recommend fixing it. Also, on 376px for both pages, there is not enough room yet to make the layout into two-column.
Hope this helps.
Marked as helpful
@faruking
Posted
@vanzasetia thank you for your inputs, I already made the design public and would take care of the rest.
@vanzasetia
Posted
@faruking Great! Now, I can see the source code. 😉