TCO932• 220
@TCO932
Submitted
Joel Guzman
@jolman009
All comments
- Joel Guzman• 250
@jolman009
Posted
Your code looks well structured. I had not seen the spacing elements in CSS like yours before. Interesting.
0 - Ejiofor John• 400
@echocode1
Submitted
What are you most proud of, and what would you do differently next time?
i completted it thought with alot of flaws.
What challenges did you encounter, and how did you overcome them?
simplifying and writing reusable code . writting lesser javascript code.witting a lesser code has always been my plan before any project but trying to bring functionality to the arrow up and down was not difficult but space consuming and i would have love learning an even more shorter reusable code.
What specific areas of your project would you like help with?
please practical input is welcome on how to write simple reusable code. aside input and review are welcome on this challenge as it is my first attempt
Joel Guzman• 250@jolman009
Posted
I hadn't seen this type of code on CSS --spacing-05: 0.125rem; --spacing-1: 0.25rem; --spacing-2: 0.5rem; --spacing-3: 0.75rem; Interesting Concept
0 - PAdamullaOsas• 390
@AdamullaOsas
Submitted
Joel Guzman• 250@jolman009
Posted
Great job with your code. It looks very smooth and well-structured.
0 - Bart Feesley• 220
@antimatterhut
Submitted
What are you most proud of, and what would you do differently next time?
This is the first project that I wrote that was reviewed by others. I recently learned about the best practice generally being no set heights/widths. I mostly ended up doing that this time, with the exception of one time where I think I was essentially forced into using a set width and height. Also, this is the first time where I feel like my JS was pretty much on point. The last few, my JS felt very long and complex, this time it was short and simple. This is my first go at web accessibility so I am open to criticism on that front.
What challenges did you encounter, and how did you overcome them?
This was a fairly easy challenge, and was pretty much all review. However this is my first go with no set heights/widths on the majority of elements, and that was challenging to wrap my head around at first.
What specific areas of your project would you like help with?
The accessibility. I don't know what I'm doing on that front.
Joel Guzman• 250@jolman009
Posted
Great job having your interactive rating app fully functional. Code looks smooth and well-structured. Thanks for sharing!
0 - yazid78• 130
@yazid78
Submitted
Joel Guzman• 250@jolman009
Posted
I previewed your site and the Total AMt per person isnt working. Check out solutions and work back to fix your code.
0 - cassiopeia• 160
@cassiopeia001
Submitted
Joel Guzman• 250@jolman009
Posted
Tested out your links and they worked perfectly. Great job!
0 - PTuna Erten• 220
@tunaerten
Submitted
What are you most proud of, and what would you do differently next time?
This project was quite enjoyable. I navigate CSS efficiently and can implement it quickly.
What challenges did you encounter, and how did you overcome them?
The coding process took slightly longer as I needed to refresh my knowledge of forms; however, I didn't encounter significant challenges. I am looking forward to taking on more complex projects.
What specific areas of your project would you like help with?
I actually wanted to create the features using ul and li, but I received a warning that I cannot place other elements inside ul and li elements. So, I converted all of them to div. Is this usage correct, or is there another way to use other elements together with ul and li elements? I welcome any advice or suggestions and would greatly appreciate your feedback.
Joel Guzman• 250@jolman009
Posted
Your code looks very clean and well-structured. Also, your input and button action links were working properly. Great job!
Marked as helpful
1 - جهاد أحمد• 270
@Gehad28
Submitted
What are you most proud of, and what would you do differently next time?
Using animations for the appearing and disappearing of the icons.
What challenges did you encounter, and how did you overcome them?
I had a problem positioning the div of the icons in the mobile view.
I used the dev tool to know the suitable numbers for the top and left margins, but I think there should be a more convenient way to do it.
What specific areas of your project would you like help with?
The positioning of the icons' dev and any feedback is welcomed :)
Joel Guzman• 250@jolman009
Posted
Your HTML, CSS, and JS layouts look very clean and well-structured. One suggestion Id make is to check the layout using the Responsibly app. The small screen output is a little off-centered. Other than that, great job with integrating javascript!
Marked as helpful
0 - AHAMMED95• 30
@AHAMMED95
Submitted
Joel Guzman• 250@jolman009
Posted
@ahammed95
Your code for this project is very well-structured and clean. I enjoyed looking at your well-structured and readable html and css files. Great job!
0 - Mohammad Ragab• 160
@E0MR
Submitted
Joel Guzman• 250@jolman009
Posted
Very smooth and well-structured code. Layout looks good and accessible. Great job!
0 - Emal• 110
@syeero7
Submitted
What are you most proud of, and what would you do differently next time?
Next time I need to work on finishing the project without taking up too much time
Joel Guzman• 250@jolman009
Posted
@syeero7 I also struggle with taking too much time on a project. However, realizing that I am new its better to take time and understand what I am doing than to rush and finish faster. Great job!
1 - Georg• 60
@GeorgStrassberger
Submitted
Joel Guzman• 250@jolman009
Posted
Great job with the layout and the tags! The code looks good and is very similiar to the example.
0
