Your page looks good @xavier.yeung. Some things to revisit;
- Run your HTML code through the W3C validator.
- There are HTML coding errors you should be aware of and address.
- Since copy/paste from codepen you can ignore the first warning and first two errors.
- Codepen provides validators for HTML, CSS and JS. Click on the chevron in the upper right of each section and then click on the respective ‘Analyze’ link.
- The one for CSS is good. Use it and address the issue(s).
- (The one for HTML misses things which is why I recommend W3C)
- Do not use the
<br> element to force line breaks or spacing. That’s what CSS is for.
- Accessibility is about being accessible to all users. Review the giving meaningful text to links lesson. For a more thorough explanation read Web Accessibility in Mind.
To help you visualize how you have your elements on your page you can temporarily add the following universal selector
border: solid 1px lightgreen
Is there a reason you’ve set the CSS preprocessor to SCSS when everything is straight CSS?
To help with responsiveness it will be better if you use relative units rather than hard coding pixel values.