rmglennon / transit-firebase

Build transit schedules with Firebase + JS, jQuery, Bootstrap, HTML, and CSS

Home Page:https://rmglennon.github.io/transit-firebase/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Feedback from review

rmglennon opened this issue · comments

index.html

  • It would be better UI in my opinion to have the form above the table, that way if there are 100 trains I don’t need to scroll all the way down to add another train.
  • The momentjs cdn doesn’t work exactly right when deployed because you need to make the protocol https not http.

transit.js

  • Your storage bucket field in your config object should have a string value there, it seems that your data is being stored but maybe not where you want it to be stored.

style.css

  • in mobile view the font-size of your table display is pushing your view out of whack.