WD-30045 - Fix Copilot comments

Author: alvaromateo

static/js/base/navigation/globalNav.ts

@@ -5,8 +5,8 @@ export function patchAllCanonicalMobileMarkup() {
   );
 
   topMobileSections?.forEach((section: Element) => {
-    const sectionButton = section.querySelector("button");
-    const sectionHref = sectionButton?.getAttribute("href");
+    const sectionLink = section.querySelector("a.p-navigation__link");
+    const sectionHref = sectionLink?.getAttribute("href");
 
     const sectionLinksList = section.querySelector("ul");
     sectionLinksList?.setAttribute("aria-hidden", "true");
@@ -28,8 +28,8 @@ function createBackButtonItem(href: string) {
   // remove the # from the href
   const ariaControls = href.slice(1);
   return createFromHTML(`<li class="p-navigation__item--dropdown-close">
-      <button href=${href} aria-controls=${ariaControls} class="p-navigation__link js-back-button">
+      <a href=${href} aria-controls=${ariaControls} class="p-navigation__link js-back-button">
         Back
-      </button>
+      </a>
     </li>`);
 }

static/js/base/navigation/listeners.ts

@@ -164,11 +164,9 @@ export const initNavigationListeners = () => {
 
         if (target.getAttribute("aria-hidden") === "true") {
           unfocusAllLinks();
-          console.log(toggle.parentElement?.classList);
           expandDropdown(toggle, target);
           navigation.classList.add("has-menu-open");
         } else {
-          console.log(toggle.parentElement?.classList);
           collapseDropdown(toggle, target);
           if (!isNested) {
             closeMenu();

static/sass/_snapcraft_p-navigation.scss

@@ -72,7 +72,7 @@
         syntax: "<percentage>";
       }
 
-      .p-navigation__nav--wrapper{
+      .p-navigation__nav--wrapper {
         display: flex;
         flex-direction: row;
         justify-content: space-between;