Martineznovo has uploaded a new change for review. Change subject: (bug 25072) Fix weirdness in collapse threads ......................................................................
(bug 25072) Fix weirdness in collapse threads The problem was the use of find(), which is fully recursive, instead of children(), which only search for the inmediate children. Since this function is called recursively for .lqt-thread-replies this prevents adding nested replies twice. Also, modified the toggleCollapse function adding '.lqt-post-wrapper,.lqt-thread-replies' to children() so it only hides and shows messages and thread replies, leaving other elements that aren't displayed anyway. This is to prevent it from also hiding the title of the entire thread, which also contains the action links, that was increasing the weirdness on pages with several threads. Change-Id: I520b0451d55f4eae2bbcf795c46f975b4b0af63b --- M jquery/jquery.thread_collapse.js 1 file changed, 5 insertions(+), 6 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/LiquidThreads refs/changes/49/36349/1 -- To view, visit https://gerrit.wikimedia.org/r/36349 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I520b0451d55f4eae2bbcf795c46f975b4b0af63b Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/LiquidThreads Gerrit-Branch: master Gerrit-Owner: Martineznovo <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
