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

Post Comments Query Loop block #34996

Open
SantosGuillamot opened this issue Sep 21, 2021 · 1 comment
Open

Post Comments Query Loop block #34996

SantosGuillamot opened this issue Sep 21, 2021 · 1 comment

Comments

@SantosGuillamot
Copy link

@SantosGuillamot SantosGuillamot commented Sep 21, 2021

Description

An advanced block that allows displaying the comments of a specific post based on different parameters and visual configurations. It should be similar to the Query Loop block.

This block is part of the Post Comments Loop block, for more context please visit this Tracking Issue or its GitHub project.

Potential Settings

  • Layout settings: It should be similar to the ones in the Query Loop.
    • It can inherit the default layout or change the width and content.
    • It could be nice to be able to select the content and width (or the padding) of the child comments.
  • Color settings: It should be the same as the Query Loop.
  • Loop settings:
    • Order by: There is a Discussion Setting to select the order of the comments. I guess we should use this by default unless they change it in the Comment Loop block.
  • List view settings. We can replicate the settings in the Query Loop:
    • Items per page: There is a Discussion Setting to select how many top level comments users want per page. I think we should respect that unless it is changed in the block itself.
    • Offset.
    • Max pages.
  • Advanced settings: We can replicate the behaviour in the Query Loop:
    • HTML element: Allow users to select if they want a <div> , <section> , <aside> HTML element.

References

  • This should be similar to the Query Loop block: Guide and GitHub.
@michalczaplinski
Copy link

@michalczaplinski michalczaplinski commented Sep 22, 2021

Could you assign the issue to me @SantosGuillamot? It seem that I cannot do it myself as a contributor. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants