Remove focus from the toggleButton when closing the sidebar with a mouse (issue 17361)
This commit is contained in:
		
							parent
							
								
									7e64f8213d
								
							
						
					
					
						commit
						6b6b7867af
					
				| @ -264,7 +264,7 @@ class PDFSidebar { | |||||||
|     this.#hideUINotification(); |     this.#hideUINotification(); | ||||||
|   } |   } | ||||||
| 
 | 
 | ||||||
|   close() { |   close(evt = null) { | ||||||
|     if (!this.isOpen) { |     if (!this.isOpen) { | ||||||
|       return; |       return; | ||||||
|     } |     } | ||||||
| @ -276,11 +276,16 @@ class PDFSidebar { | |||||||
| 
 | 
 | ||||||
|     this.onToggled(); |     this.onToggled(); | ||||||
|     this.#dispatchEvent(); |     this.#dispatchEvent(); | ||||||
|  | 
 | ||||||
|  |     if (evt?.detail > 0) { | ||||||
|  |       // Remove focus from the toggleButton if it's clicked (see issue 17361).
 | ||||||
|  |       this.toggleButton.blur(); | ||||||
|  |     } | ||||||
|   } |   } | ||||||
| 
 | 
 | ||||||
|   toggle() { |   toggle(evt = null) { | ||||||
|     if (this.isOpen) { |     if (this.isOpen) { | ||||||
|       this.close(); |       this.close(evt); | ||||||
|     } else { |     } else { | ||||||
|       this.open(); |       this.open(); | ||||||
|     } |     } | ||||||
| @ -334,8 +339,8 @@ class PDFSidebar { | |||||||
|       } |       } | ||||||
|     }); |     }); | ||||||
| 
 | 
 | ||||||
|     this.toggleButton.addEventListener("click", () => { |     this.toggleButton.addEventListener("click", evt => { | ||||||
|       this.toggle(); |       this.toggle(evt); | ||||||
|     }); |     }); | ||||||
| 
 | 
 | ||||||
|     // Buttons for switching views.
 |     // Buttons for switching views.
 | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user