Restaurant Menu Display

I Built This
1

Hi Guys,

Please provide your feedback and suggestions on the Restaurant Menu Display project that I just completed using Javascript.

https://codepen.io/Saradha3/full/BajYGaa

1 Like
2

Hello~!

The page looks very nice! I am impressed with the use of DOM manipulation :slight_smile:

Also, this description made me lol

image
image525Ă—563 80.6 KB

1 Like
3

Thank you so much :slight_smile:

God I actually forgot to change the description of Buttermilk Pancakes. I just copied it from another page. Infact I dont even know the meaning of this :joy:

1 Like
4

It’s not bad and looks nice. A few suggestions:

  • The menu at the top should have an indicator to show which one is active
  • Narrow the browser down until it shows only one column of food. Now manually increase the text size. You will see that at some point the menu items at the top start to disappear off the right side of the page and there is a horizontal scroll bar
  • Don’t skip heading levels. The current h3 should be an h1 and the h4’s should be h2’s
  • Actually, you currently have two h4’s for each item. That should probably change. The name of the item should be an h2. I’m not sure what the “Rs” number represents but it should probably not be the same level of heading as the item name (if it should even be a heading?)
  • You might consider using a figure/figcaption for the the items
  • Technically, all the images need alt text. If you don’t want to add the alt text then set alt="" and and aria-hidden=“true” for screen readers
2 Likes
5

Thank you so much for your valuable feedback :slight_smile:

I have added alt to the images now. I will work on your other suggestions.