Fully Responsive Page on All Devices Using Boostrap 5 and CSS

Solution retrospective
Hi Community! I have completed this challenge but I had to use too many CSS media queries to make it responsive on all devices. I used Bootstrap 5 too but didn't fix the issues all the time. Kindly have a look at the code and suggest to me what can I improve. Thanks.
Please log in to post a comment
Log in with GitHubCommunity feedback
- @hardy333
hey, website looks good. nice job.
Few suggestions:
-
try to align huddle logo and big showcase image correctly on left size, for that I suggest you to use one container/wrapper div which will have some margin from left, your logo and image will be exactly the left side of that container.
-
Try to add box shadow on cta button and maybe some more hover effects on it...
-
In terms of using bootstrap, it is ok but you don't need bootstrap here, it will make your life just harder, this is very simple layout use instead vanilla css ...
-
- @pikapikamart
Hey, great work on this. Layout looks good both desktop and mobile. The responsiveness is fine as well.
I second to what hardy said about bootstrap. You don't need that in here. Go with just pure vanilla css or scss. That would make you much greater in stylings, do not go using framework.
Some other suggestions would be:
- The
alt
text for the website logo should bealt="huddle"
. The website logo already have the text on it, better use it as thealt
value. - Always have a
main
element on a webpage. This helps user navigate properly within your site. On this, thediv
that captures all the layout should be themain
tag. - The
register
should usea
tag instead ofbutton
. - The
register
lacks a visual indicator on it. If you try usingtab
key in your keyboard, you won't see anything that indicates you are within the element. Use some stylings in the:focus-visible
selector of that element. Apllying aoutline
orborder
orbox-shadow
. - The social media links should be inside
ul
element, each with their ownli
element, since those are list of links. - The
a
tag that wrap the social media link should havearia-label
on it. For example:
<a aria-label="facebook"> { the icon in here } </a>
This way, users will know where this link will take them.
Aside from those, great work.
- The
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord