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

Weird HTML insertion on new post #7432

Closed
RoiEXLab opened this issue Mar 7, 2019 · 2 comments
Closed

Weird HTML insertion on new post #7432

RoiEXLab opened this issue Mar 7, 2019 · 2 comments
Assignees
Labels
Milestone

Comments

@RoiEXLab
Copy link
Contributor

RoiEXLab commented Mar 7, 2019

NodeBB version: 1.11.1, but should also be the case on master
Theme: Persona
Not sure if this is an issue with persona or core, but I'm posting this here anyways for the visibility.
I forgot to make a screenshot of this very rare occurrence but here's my theory of what I think is going on:
There is this setting for the persona theme, that allows people to hide the "Last post on category view"
Screenshot Settings
Which, as the name implies hides this little box next to the categories:
Screenshot Teaser
There's also this code that refreshes this teaser preview whenever there's a new post.
It looks like this refresh is still happening even if there is no teaser in the first place.
So the code tries to replace a non-existing teaser. This wouldn't be a problem, however it seems that the used CSS selector still finds a valid tag, and just places the teaser inside this wrong tag, which then appears formatted badly in the middle of the category page, next to the category of the post.

I'm sorry that I can't provide a screenshot right now, but at least I'm pretty confident that my explanation is fairly accurate.

@barisusakli barisusakli self-assigned this Mar 8, 2019
@barisusakli barisusakli added the bug label Mar 8, 2019
@barisusakli barisusakli modified the milestones: 1.12.0, 1.12.1 Mar 8, 2019
@barisusakli
Copy link
Member

@RoiEXLab you can fix this by setting number of recent replies to 0 in the ACP for all categories. Or wait for the next release.

@RoiEXLab
Copy link
Contributor Author

RoiEXLab commented Mar 8, 2019

Thanks 👍

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

2 participants