Latest comments
- @Kareemhesham2005@jones9411
Not a problem but instead of having different pages for everything which slows down your website a lot, and the constant loading of new pages is a bit jarring, for example instead of having three HTML pages for the Destination page you could have had just one and used javascript to dynamically alter the page content.
Marked as helpful - @ArtemVMV@jones9411
hi @ArtemVMV, you attached the background images to the HTML selector in the CSS, try instead attaching them to the body tag instead.
Marked as helpful - @st3wn@jones9411
Hi @st3wn, you could add something like " box-shadow: 0px 3px 12px 0px rgba(0, 0, 0, 0.26); " to the .block in your css file, should apply to all for of your blocks this way.
You could also use this to learn more about making box shadows
https://html-css-js.com/css/generator/box-shadow/ - @Djokaras@jones9411
hi @djokaras,
you have put the container width at 75%, this will be true on smaller screens as well, so the content within will over flow onto the rest of the page. try setting a min-width first then set a width that fits the content after. Also you need to use the background no-repeat fot the background images.
Marked as helpful - @AleksandarV91@jones9411
Hi, it appears that when you load into the page that the error message is already there, you should add the eventlistener to the on-click of request access button. This way it's only when the form button is clicked will the error show itself. e.g button is clicked => check if there is a valid email => if not show error message => if there is show pop up message. This is the work flow I will implement for this challeng. Hope this helps
- @Danny-for@jones9411
HI @danny-for,
Good job on the solution, I've just done this one myself as well. The advice I was given was to use more semantic HTML tags like main, section, header, etc.
Also maybe use some more padding/margin to add a little bit more space between elements
Well done, Carl
Marked as helpful