@SignetOHara
Submitted
Please do let me know if there are any areas of my code that can be simplified. Also keen on practicing to make sure my websites are accessible so please recommend anything I can do to improve that too.
Looking to hire developers?
@LukZap
@SignetOHara
Submitted
Please do let me know if there are any areas of my code that can be simplified. Also keen on practicing to make sure my websites are accessible so please recommend anything I can do to improve that too.
@LukZap
Posted
Hey, I made pull request with changes I'd do to js part of project. You can merge it if you like.
https://github.com/SignetOHara/FrontendMentor-IntroComponent/pull/1
@safruk
Submitted
I have used 'min-width' media queries for the first time. Is the implementation all right?
For the colors I have used 'rgba()' value for the 'color' attribute.Is there a way to target hsl/hsla values in a way similar to how I have used rgba here?I tried doing targeting the hsla values but unfortunately it didn't work.
Any other feedback/criticism would be wecome :)
@LukZap
Posted
Hi,
For colors you can define them in your _variables.scss
like that: $ColorPrimary: hsl(179, 62%, 43%)
, and then just import it to your for ex. _grid-component.scss
like that: @import './../abstracts/variables';
And use it:
&__section-info {
background-color: $ColorPrimary;
...other styles...
}
@gur22-09
Submitted
any feedback on the form error display or anything else would be appreciated. The way i did it, is not the optimal solution.
@LukZap
Posted
hi, it will be nice to reset the validation message every time the button is clicked because now they stack on each other. You can try adding a div element below input and in click handler at the beginning set it's innerHtml = '', and then when you check email regex set it's innerHtml to validation message.
@unkletayo
Submitted
I have issues with flexbox at some point in styling navbars, please I need tutorials on it
@LukZap
Posted
Hi, I made pull request to your repo with some help. If you need tutorial I think quite nice is: https://css-tricks.com/snippets/css/a-guide-to-flexbox/ and it's nice to make some experiments by yourself with div containing different elements to get what's going on.