jsnbuchanan/crowd-funder-for-time-pwa
1
0
Fork 1
forked from trent_larson/crowd-funder-for-time-pwa
Code Issues Pull Requests Projects Releases Wiki Activity

fix: remove duplicate Advanced heading and improve UX

Browse Source 
- 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
This commit is contained in:
Matt Raymer
2025-05-15 03:37:01 -04:00
parent 29607f4e58
commit 5a4bf99534
2 changed files with 299 additions and 2487 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2780
package-lock.json generated
View File

File diff suppressed because it is too large Load Diff

6
src/views/AccountViewView.vue
View File

@@ -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!