Removes H1 from nav bar
Accessibility improvement, better for screen readers to have h1 be the content of the page as opposed to the application / brand name. Rest of the nav bar to be dealt with at a later date.
This commit is contained in:
		
							parent
							
								
									d0abf2b324
								
							
						
					
					
						commit
						133d8b10ab
					
				| @ -274,6 +274,7 @@ export class App extends LiteElement { | ||||
|         > | ||||
|           <div> | ||||
|             <a | ||||
|               class="text-sm hover:text-neutral-400 font-medium" | ||||
|               href=${homeHref} | ||||
|               @click=${(e: any) => { | ||||
|                 if (isAdmin) { | ||||
| @ -281,10 +282,9 @@ export class App extends LiteElement { | ||||
|                 } | ||||
|                 this.navLink(e); | ||||
|               }} | ||||
|               ><h1 class="text-sm hover:text-neutral-400 font-medium"> | ||||
|                 ${msg("Browsertrix Cloud")} | ||||
|               </h1></a | ||||
|             > | ||||
|               ${msg("Browsertrix Cloud")} | ||||
|             </a> | ||||
|           </div> | ||||
| 
 | ||||
|           ${isAdmin | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user