Well an empty string shouldn't crash
tag.content = tag.content.replace(/[&<>'"]/g, function(tag) { return escapeList[tag] || tag; });Make sure you are on latest on both nodebb and the lavender theme. Are you still getting any crashes?
Pagination settings don't take
-
After saving and refreshing
-
This is a bug. Report it on Github please:
-
@dangeruss @PitaJ Don't think that's a bug. User settings won't let you use a number bigger than the number in ACP -> Settings -> Pagination.
-
@pichalite said in Pagination settings don't take:
@dangeruss @PitaJ Don't think that's a bug. User settings won't let you use a number bigger than the number in ACP -> Settings -> Pagination.
Sounds like a warning needs to appear if you go higher than that value!
-
@teh_g Yes. It's a UX issue. There are other issues related to ACP/user settings for infinitescroll/pagination that also needs to be looked at.
-
@teh_g said in Pagination settings don't take:
@pichalite said in Pagination settings don't take:
@dangeruss @PitaJ Don't think that's a bug. User settings won't let you use a number bigger than the number in ACP -> Settings -> Pagination.
Sounds like a warning needs to appear if you go higher than that value!
Wait so if I set a setting higher than that in ACP settings then it should work?
Edit. Seems to have worked. Would be nice to have a max setting in ACP, so that the default settings are not also the max.
