Skip to main content
Code Review

Return to Question

Commonmark migration
Source Link

#Server-side

Server-side

##./server.js

./server.js

##./lib/chat_server.js

./lib/chat_server.js

#Client-side

Client-side

##./public/index.html

./public/index.html

##./public/js/chat.js

./public/js/chat.js

##./public/js/chat_ui.js

./public/js/chat_ui.js

#Server-side

##./server.js

##./lib/chat_server.js

#Client-side

##./public/index.html

##./public/js/chat.js

##./public/js/chat_ui.js

Server-side

./server.js

./lib/chat_server.js

Client-side

./public/index.html

./public/js/chat.js

./public/js/chat_ui.js

Notice removed Draw attention by Community Bot
Bounty Ended with no winning answer by Community Bot
Notice added Draw attention by Phrancis
Bounty Started worth 200 reputation by Phrancis
Tweeted twitter.com/StackCodeReview/status/838086323672055809
added 124 characters in body
Source Link
Phrancis
  • 20.5k
  • 6
  • 69
  • 155

Overall I think it reads a lot easier like this, I again extracted all those in-line functions to their respective functions to make them easier to document and maintain. This is my first experience with jQuery so I feel especially sure that this part of the code could be improved a lot still.

Overall I think it reads a lot easier like this, I again extracted all those in-line functions to their respective functions to make them easier to document and maintain.

Overall I think it reads a lot easier like this, I again extracted all those in-line functions to their respective functions to make them easier to document and maintain. This is my first experience with jQuery so I feel especially sure that this part of the code could be improved a lot still.

add screenshot
Source Link
Phrancis
  • 20.5k
  • 6
  • 69
  • 155

This is what it looks like:

chat

This is what it looks like:

chat

edited tags
Link
Phrancis
  • 20.5k
  • 6
  • 69
  • 155
Loading
Source Link
Phrancis
  • 20.5k
  • 6
  • 69
  • 155
Loading
lang-js

AltStyle によって変換されたページ (->オリジナル) /