Dine Restaurant with HTML, CSS, JS

Community feedback

• 

  Raymart Pamplona•16,040

  @pikapikamart

  Posted about 4 years ago

  Hey great work but there are issues that I saw. First, your css declarations, we can't preview the desktop state since I saw in your css in your repository, you declared in the @media (min-width: 1440px) the styling that is supposed for desktop resolution right. But the thing is that, since you made it mobile first as far as I can tell, setting the desktop styling for min-width: 1440px, will only allow user who have their desktop width of 1440px wider and any lower than that, will only see the mobile version state and you don't want that. Maybe adjusting your min-width like 1024px something like that would be awesome. Your declaration of javascript. I see that you only created one js script and the issue with is your other html, you made it dependent on it, both your .html script is written on only one, that is why an error is prompting saying that there are variables who are null, since the variables element, is in another .html file. It would be better that you create a seperate js files right for both you .html files so that error wont be prompted. For for form reservations, it is pretty I think, do you mean the js for it or the stylings and that event-list? Could you be more specific about it? Well that are just some recommendations^^

  12