Latest solutions
News Homepage Built Using Vite, SASS, Typescript
Submitted 3 months agoI would appreciate accessibility checks and feedback
FAQ Accordion Website Built Using VanillaJS, TypeScript, Gulp and SASS
#gulp#sass/scss#typescript#accessibilitySubmitted 3 months agoResponsive, Accessible Interactive Rating Component Using Typescript
#gulp#sass/scss#typescriptSubmitted 4 months agoAny areas where you notice anything off/wrong
Frontend Quiz App Using VanillaJS, NodeJS, Express, and MongoDB
#mongodb#node#sass/scss#expressSubmitted 4 months agoI would like help with the
index.ts
,app.js
, anddb.js
Thanks
Password Generator Using Typescript and SASS
#typescript#sass/scssSubmitted 4 months agoI would like a review of my index.ts file. I would appreciate any feedback on it
Latest comments
- @chanwinharold@Achigyus
Hey mate.
Great work on the site, it looks really great. I just have one little tip
You should set a max-width on the header, main, and footer tags along with a margin of
0 auto
to center everything.Marked as helpful - @SimonHickling@Achigyus
Hey Simon.
Thanks for your feedback on my solution. I thought to return the favor by helping you review your solution.
Here are some tips to improve the design.
-
You need to properly center the contact card, this can be achieved by using flexbox or grid
-
You need to set pointer styles on interactive elements such as the inputs, buttons, etc. You can set it using
cursor: pointer
-
It would also be a good idea to add some padding to the inputs and textarea to make it look better.
These are some of the things I noticed. Thanks
Marked as helpful -
- @Smart-Ace-DesignsWhat are you most proud of, and what would you do differently next time?
I used Astro Actions for form validation. I had already used this for another FEM challenge, but it was nice to refresh my skills on this.
I try to only use AI to "assist me" when I have questions and not completely write the code for me.
What challenges did you encounter, and how did you overcome them?Following proper accessibility solutions. Not difficult...just something to make sure it is not overlooked. I used AI to help with aria properties when I was not sure what to include.
What specific areas of your project would you like help with?Nothing at this time.
@AchigyusHey Smart Ace
Great work on the design.
I just noticed a few things.
-
It would be good to set an event listener on the inputs so that when the user fills out the fields he/she missed out, the error messages are cleared
-
There are also some inconsistencies in your use of
cursor: pointer
. Some elements are not completely wrapped -
Your error message for email validation reads
A valid email address name is required
, I think it would be better to useA valid email address is required
.
All in all, a great solution
Marked as helpful -
- P@Coder-Liz@Achigyus
Absolutely a great solution, Liz.
I noticed that when using just the tab key to navigate the website, there was no outline on the buttons, so it seemed like it wasn't working.
It was a small fix, and I forked your repo and created a pull request.
You can check the pull request here
- P@danielmrz-devWhat are you most proud of, and what would you do differently next time?
🛸 Hello FEM Community! I'm Daniel and this is my solution for this challenge! 😊
🛠️ Built with:
- Angular 🅰️
- SASS 🎨
- TypeScript 🔷
- BEM Notation 🅱️
- Mobile first workflow approach 📲
My first GURU level project 😊
If you have any suggestions on how I can improve this project, feel free to leave me a comment!
Feedback welcome 😊
Designo Multi-Page Website built with Angular + SASS 🅰️
#angular#angular-material#sass/scss#typescript#rxjs@AchigyusGreat work, buddy. Congrats on completing your first guru level project
- P@Coder-Liz@Achigyus
Hey @Coder-Liz,
A really great solution; all buttons are accessible and work well. The UI looks good as well.
The only thing I noticed is that it would be nice to add some cursor styling to the buttons apart from just the background change on hover.
You can add that with the code below:
.rating__submit-btn, .rating__number-btn { cursor: pointer; }