@Ayako-YokoeSubmitted 9 days ago
P
Sanya
@Sanya-ZgAll comments
- P@Sanya-ZgPosted 8 days ago
You can add an "input" event listener for each input to instantly update the error message. But overall it's great.
1 - @ramiadiSubmitted 22 days agoWhat are you most proud of, and what would you do differently next time?
I actually used more elements in html instead of using the <div> tag!
What challenges did you encounter, and how did you overcome them?It took some time with the elements in the html, but chatgpt and the internet helped me
What specific areas of your project would you like help with?is my aria labels good? for people who have a hard time hearing (using applications for reading the website) is the code friendly towards them?
P@Sanya-ZgPosted 20 days agoSome elements are a bit different from the template. But overall it's good.
0 - @shadowbanksSubmitted 25 days agoWhat specific areas of your project would you like help with?
I'm open to any suggestions.
Thank you for your time :))
- P@nashrulmalikSubmitted 3 months ago
- @muiruri3000Submitted 6 months agoWhat are you most proud of, and what would you do differently next time?
being able to refactor the initial code I built with.
What challenges did you encounter, and how did you overcome them?returning objects from a function and some properties having no values was a bit of an issue but I quickly got a round that by returning properties with empty values and then later handling such scenarios of empty values.
What specific areas of your project would you like help with?I could use a bit more higher order functions
- @muiruri3000Submitted 6 months agoWhat are you most proud of, and what would you do differently next time?
being able to refactor the initial code I built with.
What challenges did you encounter, and how did you overcome them?returning objects from a function and some properties having no values was a bit of an issue but I quickly got a round that by returning properties with empty values and then later handling such scenarios of empty values.
What specific areas of your project would you like help with?I could use a bit more higher order functions
- P@Richard1876Submitted 3 months agoWhat challenges did you encounter, and how did you overcome them?
i had a hard time putting all the pieces together with JS at first
- @tris6Submitted 4 months agoWhat are you most proud of, and what would you do differently next time?
Having a grid of six similar cards, it was very satisfying to set properties and watch all update uniformly at the same time. Cascading felt very elegant.
I didn't invest much time getting the aesthetics picture perfect, as I find the trial-and-error time-consuming without much learning benefit. For example, I didn't bother to align the category icons per preview.
What challenges did you encounter, and how did you overcome them?I hit a roadblock highlighting active state for the text to select timeframe. My JS function resets all colours back to the default colour, before highlighting the active timeframe. But, because I set properties directly, specificity was greater, so onhover didn't play nicely with active. As a fix, the JS adds/removes a highlight class rather than sets properties directly, making all highlighting operate on the same level.
The fetch URL to JSON file worked fine in localhost, but was hitting a 404 on GitHub. To move up one folder, I used ./ rather than ../ - I don't know why this worked, but I saw another student do it, and it works.
For highlighting the ellipse, I simply made a copy of the SVG and changed fill colour to white. So hover changes from blue ellipse to white ellipse. I'm very tired of working with SVG fill properties, so happy to have this workaround.
What specific areas of your project would you like help with?I'm very keen to see other solutions. I suspect my code is overly-verbose from being new to many of these problems.
For positioning the colored strips overlapping the content, my properties became very pixel-perfect rather than intrinsic.
My JS also selects every timefield using its' own individual ID. So I have 15 lines of just .getElementById. I wonder if there's a more elegant way to map ID to variables.
- @tris6Submitted 4 months agoWhat are you most proud of, and what would you do differently next time?
Having a grid of six similar cards, it was very satisfying to set properties and watch all update uniformly at the same time. Cascading felt very elegant.
I didn't invest much time getting the aesthetics picture perfect, as I find the trial-and-error time-consuming without much learning benefit. For example, I didn't bother to align the category icons per preview.
What challenges did you encounter, and how did you overcome them?I hit a roadblock highlighting active state for the text to select timeframe. My JS function resets all colours back to the default colour, before highlighting the active timeframe. But, because I set properties directly, specificity was greater, so onhover didn't play nicely with active. As a fix, the JS adds/removes a highlight class rather than sets properties directly, making all highlighting operate on the same level.
The fetch URL to JSON file worked fine in localhost, but was hitting a 404 on GitHub. To move up one folder, I used ./ rather than ../ - I don't know why this worked, but I saw another student do it, and it works.
For highlighting the ellipse, I simply made a copy of the SVG and changed fill colour to white. So hover changes from blue ellipse to white ellipse. I'm very tired of working with SVG fill properties, so happy to have this workaround.
What specific areas of your project would you like help with?I'm very keen to see other solutions. I suspect my code is overly-verbose from being new to many of these problems.
For positioning the colored strips overlapping the content, my properties became very pixel-perfect rather than intrinsic.
My JS also selects every timefield using its' own individual ID. So I have 15 lines of just .getElementById. I wonder if there's a more elegant way to map ID to variables.
- @mehedihasan1179Submitted 4 months ago
- @andersongomes001Submitted 4 months ago
- P@wecaxSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
N/A
What challenges did you encounter, and how did you overcome them?N/A
What specific areas of your project would you like help with?N/A
- @JhodelDatilesSubmitted 5 months agoP@Sanya-ZgPosted 5 months ago
Hi! The first thing that catches your eye is not the same font as in the layout. The outer indents are a little small, the indents between the elements inside are not exact. Pay attention to the meticulous execution of details. The grid is perfectly executed as in the layout. Good luck
0 - @BunchydoSubmitted 5 months agoWhat are you most proud of, and what would you do differently next time?
I am most proud of the fact that it is taking me less time to code the websites than when I first started. Although far from efficient code, I am still proud of how far I have come.
What challenges did you encounter, and how did you overcome them?The challenges I encountered were in the selection of whether to use rem, em, px, vh/vw or % for different elements to make coding the website responsiveness easier for me.
What specific areas of your project would you like help with?I would like help to know when to use rem, em, px, vh/vw or % for different elements to make coding the website responsiveness easier for me.
P@Sanya-ZgPosted 5 months agoIt is necessary to work with indents between elements. Also the thickness of the headings. Place the icons on the right. There are several options to achieve this. Other than that, the whole project is well done.
0 - @Mehdi-MISubmitted 6 months agoP@Sanya-ZgPosted 5 months ago
A different font is needed for the title and the new price. The indents between the rows are a bit inaccurate. You can be more precise with Figma. In general, good work.
0 - @kthgrmSubmitted 5 months agoWhat challenges did you encounter, and how did you overcome them?
aligning the table data
P@Sanya-ZgPosted 5 months agoGood job! For a more accurate result, use Figma. For greater table flexibility, use the Grid. Good luck!
0 - @Jamal-DigitalSubmitted 6 months agoWhat are you most proud of, and what would you do differently next time?
I'm proud i improved my responsive design
P@Sanya-ZgPosted 6 months agoGood work. Good luck with your next projects.
Marked as helpful0 - @barahawSubmitted 6 months agoP@Sanya-ZgPosted 6 months ago
You need to change the font to the recommended one. Increase the indentation from the top and bottom. The button is too big. Increase the margin on the sides of the text to 24px. The font of the h1 header should be slightly larger.
0
