@alidhaini
Latest solutions
Interactive rating component
Submitted 10 months agoWhen I have two different html pages how do I handle JavaScript files? I use main.js for index.html and using html script elements for the other html pages.
Time Tracking Dashboard
Submitted 10 months agoI ended up using a ton of if statements to properly assign the data to its correct place. Is there a more efficient way of doing this?
Latest comments
- @matt2282
Good Job!
A few things to note: *On the nav bar on the mobile screen, you cannot exit the nav bar with the keyboard only. *The background color on the design is off-white, you used white on your solution *The ability to select the hyperlinks on the page with a keyboard is missing.("Hydrogen VS Electric cars", "Reviving Retro PCs", ect.) *If you resize the page to larger screen sizes the main image gets stretched out. You could either limit the size of the grid or make the grid scale up to larger screens.
- @samuel-adu@matt2282
Very Good!
It might be more intuitive to have the query type buttons work when you click anywhere in the box, and to also display a pointer cursor.
The consent checkbox is overlapping with the custom image when it is checked.
- @konradbaczyk@matt2282
I like the animation on the buttons and the accordion functions as expected.
A few bugs and issues I encountered:
- When on mobile and you expand the accordion all the way out on all four questions the top of the page can start to get cut off.
- Your page does not have any support for keyboard only navigation.
Marked as helpful - P@kaamiikWhat are you most proud of, and what would you do differently next time?
Using Astro for the first time in a project
What challenges did you encounter, and how did you overcome them?HTML structure and do this project based on making component was challenging
What specific areas of your project would you like help with?I think accessibility issues is the most important part and html structure is important too.
@matt2282Good job!
My understanding of accessibility is that your page needs to meet a few criteria: -alternate text on images: you provided an alt text on the image on the thank you page. -navigation with the tab key on the keyboard: the tab button does not select the rating buttons on the main page. -using the correct semantic html elements -ensuring a screen reader can read the content on the page and its convey its context.
There is more to it than this, but this is the basics I have learned so far.
- @ikethedevWhat are you most proud of, and what would you do differently next time?
I am proud that I wrote this more modular and I believe I did a solid job on naming function descriptive names
What challenges did you encounter, and how did you overcome them?A few challenges I faced was writing a function that updated the object in a way that wasn't overly complicated
What specific areas of your project would you like help with?Still working on pixel perfection!
@matt2282Good job on the overall layout of the app, but I would recommend giving this another shot. I can't seem to get your calculator to give any values or at least when it does output some values they are not correct.
The grey "bill" and "number of people" and "custom" divs do not go 100% of the container and they don't line up with everything else. The text in placeholder for Custom also isn't centered.
Some additional features you may consider adding: -border around the custom tip, bill, number of people and amount to know you have it selected. -when you press reset it may be preferential to set 0 as the placeholder and not have it be an actual zero. Additionally, having the reset unselect the selected tip value if you prefer. -error checking for zero and subsequent error message -allow only positive integers as the input -greying out the reset button after it is pressed.
- @antimatterhutWhat are you most proud of, and what would you do differently next time?
I am very proud of the fact that I could do this at all.
What challenges did you encounter, and how did you overcome them?I have never worked with async/await or promises or none of that. I didnt even know why they were used or what they did until now.
What specific areas of your project would you like help with?Optimally, I'd like to condense the js code into just 2 big functions instead of 6.
@matt2282Good job! I don't have much to say except that you forgot the hover state on the ellipsis. It is supposed to turn white when you hover over it.
Marked as helpful
