fix: remove duplicate Advanced heading and improve UX

- Remove redundant Advanced heading from advanced settings section\n- Make clickable text more descriptive (Show/Hide Advanced Settings)\n- Add cursor-pointer class for better UX\n- Make section heading screen-reader only for accessibility

src/views/AccountViewView.vue

@@ -448,13 +448,13 @@
     <!-- id used by puppeteer test script -->
     <h3
       id="advanced"
-      class="text-blue-500 text-sm font-semibold mb-3"
+      class="text-blue-500 text-sm font-semibold mb-3 cursor-pointer"
       @click="showAdvanced = !showAdvanced"
     >
-      Advanced
+      {{ showAdvanced ? 'Hide Advanced Settings' : 'Show Advanced Settings' }}
     </h3>
     <section v-if="showAdvanced || showGeneralAdvanced" id="sectionAdvanced" aria-labelledby="advancedHeading">
-      <h2 id="advancedHeading" class="text-blue-500 text-sm font-semibold mb-3">Advanced</h2>
+      <h2 id="advancedHeading" class="sr-only">Advanced Settings</h2>
       <p class="text-rose-600 mb-8">
         Beware: the features here can be confusing and even change data in ways
         you do not expect. But we support your freedom!