Refactor login/logout URL generation in BaseForm

Author: ruibaby

packages/comment-widget/src/base-form.ts

@@ -73,17 +73,15 @@ export class BaseForm extends LitElement {
     );
   }
 
-  get loginUrl() {
-    const parentDomId = `#comment-${[
-      this.group?.replaceAll('.', '-'),
-      this.kind,
-      this.name,
-    ]
+  get parentDomId() {
+    return `#comment-${[this.group?.replaceAll('.', '-'), this.kind, this.name]
       .join('-')
       .replaceAll(/-+/g, '-')}`;
+  }
 
+  get loginUrl() {
     return `/login?redirect_uri=${encodeURIComponent(
-      window.location.pathname + parentDomId
+      window.location.pathname + this.parentDomId
     )}`;
   }
 
@@ -140,7 +138,9 @@ export class BaseForm extends LitElement {
       )
     ) {
       try {
-        window.location.href = '/logout';
+        window.location.href = `/logout?redirect_uri=${encodeURIComponent(
+          window.location.pathname + this.parentDomId
+        )}`;
       } catch (error) {
         console.error('Failed to logout', error);
       }
@@ -170,13 +170,6 @@ export class BaseForm extends LitElement {
     </div>`;
   }
 
-  onContentInput(e: Event) {
-    const target = e.target as HTMLTextAreaElement;
-    // reset height to auto to make sure it can grow
-    target.style.height = 'auto';
-    target.style.height = `${target.scrollHeight}px`;
-  }
-
   onKeydown(e: KeyboardEvent) {
     if (e.key === 'Enter' && (e.metaKey || e.ctrlKey)) {
       const form = this.shadowRoot?.querySelector('form');