search

mjayjones13 / beans

0 stars 0 forks source link

Client: UX - Create Notifications and Header UI Controls (fixes #18) #27

Closed joshcox closed 7 years ago

joshcox commented 7 years ago

Brought App back to a working state; created Header and Notifications UI Control; ensured that original DOM markup was retained using the new classes; removed skeleton HTML that lived in beans.html (fixes #18)

joshcox commented 7 years ago

For some reason our app.js was throwing an error when running npm start, so I fixed that issue so you can start up the app now and see it @ http://localhost:3000

Also - I commented the BeansView stuff out for now and used our UI Controls to create the DOM. See the index.html file for that stuff. When we create the Content and BeansView UI Controls, that will get removed again. I just wanted to verify that our stuff was working as expected.

In addition to that, I did some updates on the Menu class that you'll want to check out. That was my bad - I've forgotten a lot of the jQuery syntax and I missed some of our mistakes.

mjayjones13 commented 7 years ago

Looks good to me other than that minor comment correction