thoughtbot / carnival

An unobtrusive, developer-friendly way to add comments
MIT License
499 stars 30 forks source link

Revert fix for Firefox #254

Closed pbrisbin closed 9 years ago

pbrisbin commented 9 years ago

The fix for Firefox breaks Chrome, and didn't really fix Firefox anyway.

The reason Firefox was broken is because we make a synchronous request for user details. We should not be doing this anyway. The decision was made to revert the Firefox "fix" and focus on making the user request async instead.

jferris commented 9 years ago

Cool.