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.

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

Need Reply class #276

Open
fracture91 opened this issue Jun 24, 2012 · 0 comments
Open

Need Reply class #276

fracture91 opened this issue Jun 24, 2012 · 0 comments
Labels
Milestone

Comments

@fracture91
Copy link
Owner

The old Reply class allowed you to get info about a reply and was used in a lot of places. We'll need this before we can do much with replies.

I'm tempted to say that this should use my MVC thing I made, and autorefresh should parse things out into a Model that we give to a View to avoid security problems, but that would just be an immense pain in the ass. Wouldn't help much once we have to poop innerHTML somewhere for sigs and actual reply content :P. I guess to solve that problem, we might pass our autorefresh stuff through Firefox's sanitizer thing... whatever.

Anyway, this class needs to be redone, but only the information we need should be retrievable.

https://github.com/fracture91/ignpp/blob/2.3.0/content/replies.js

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant