Fix scrolling to reply via 'reply' as near parameter

This commit is contained in:
Alexander Skvortsov 2020-10-16 12:04:45 -04:00
parent 2df4a1a4a8
commit 086c06145c

View File

@ -15,7 +15,7 @@ function getDiscussionIdFromSlug(slug: string | undefined) {
* in the same discussion. * in the same discussion.
*/ */
export default class DiscussionPageResolver extends DefaultResolver { export default class DiscussionPageResolver extends DefaultResolver {
static scrollToPostNumber: number | null = null; static scrollToPostNumber: string | number | null = null;
makeKey() { makeKey() {
const params = { ...m.route.param() }; const params = { ...m.route.param() };
@ -28,7 +28,7 @@ export default class DiscussionPageResolver extends DefaultResolver {
onmatch(args, requestedPath, route) { onmatch(args, requestedPath, route) {
if (route.includes('/d/:id') && getDiscussionIdFromSlug(args.id) === getDiscussionIdFromSlug(m.route.param('id'))) { if (route.includes('/d/:id') && getDiscussionIdFromSlug(args.id) === getDiscussionIdFromSlug(m.route.param('id'))) {
DiscussionPageResolver.scrollToPostNumber = parseInt(args.near); DiscussionPageResolver.scrollToPostNumber = args.near === 'reply' ? 'reply' : parseInt(args.near);
} }
return super.onmatch(args, requestedPath, route); return super.onmatch(args, requestedPath, route);