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

Comment reply to entries #98

Open
mjangda opened this issue Jun 22, 2013 · 3 comments · May be fixed by #103
Open

Comment reply to entries #98

mjangda opened this issue Jun 22, 2013 · 3 comments · May be fixed by #103

Comments

@mjangda
Copy link
Member

mjangda commented Jun 22, 2013

As a user, I want to be able to interact with a Liveblog publisher. Since Liveblog entries are stored as comments, we can store replies to each entry as a nested comment and maybe use core's reply-to-comments to provide an inline reply experience.

The focus should still be on the Liveblog entries, but users should be able to dive into other viewer comments if they like.

@westonruter
Copy link
Contributor

I suppose that visitors should be restricted only adding comment replies to liveblog comments, or to other liveblog comment replies. In other words, should users be prevented from adding top-level comments?

@mjangda
Copy link
Member Author

mjangda commented Jun 26, 2013

Top-level comments should stay functional.

I think we may need to introduce the ability to enable/disable commenting on Liveblog entries, though (separate from the top-level comments).

@keoshi can help with mocks on how this might look/work, if needed.

@westonruter westonruter linked a pull request Jul 9, 2013 that will close this issue
@westonruter
Copy link
Contributor

@mjangda Regarding the ability to enable/disable commenting on Liveblog entries, see: xwp@a3a713e

Liveblog commenting closed

closed comments metabox state

Liveblog commenting open

open comments metabox state

Comments on post not allowed

if comments on post are closed entirely

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

Successfully merging a pull request may close this issue.

4 participants