Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[streaming] switch streaming junk from atmosphere to whitespace #516

Closed
jfarcand opened this issue Jul 16, 2012 · 1 comment
Closed

[streaming] switch streaming junk from atmosphere to whitespace #516

jfarcand opened this issue Jul 16, 2012 · 1 comment
Labels
1.0.0 Will not Fix

Comments

@jfarcand
Copy link
Member

Both SSE and Android supports already use whitespace, so let's change the current atmosphere mode to whitespace as well. That means no more <--- Atmosphere ....> junk but instead whitespace, which are trimmed by atmosphere.js by default.
jfarcand added a commit that referenced this issue Jul 16, 2012
@jfarcand
Fix for #516 [streaming] switch streaming junk from atmosphere to whi…
0ae35b1 
…tespace
@jfarcand jfarcand reopened this Jul 16, 2012
@jfarcand
Copy link
Member Author

This changes breaks IE. Rolling back.

jfarcand added a commit that referenced this issue Jul 16, 2012
@jfarcand
Rollback fix for #516 as it breaks IE. Instead add a new API for fixing
8c09820 
#400 padding issue
jfarcand added a commit that referenced this issue Jul 16, 2012
@jfarcand
Rollback for #516
01a3489
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1.0.0 Will not Fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jfarcand