fx-compat: <split-menu-button>: Use margin instead of absolute height
And add space to the right of the button text. Should look right on Linux now. Fixes #3041
This commit is contained in:
		
					parent
					
						
							
								ab2e163234
							
						
					
				
			
			
				commit
				
					
						d9b16c15f7
					
				
			
		
					 2 changed files with 6 additions and 5 deletions
				
			
		|  | @ -71,6 +71,9 @@ | ||||||
| 			let hiDPISuffix = window.devicePixelRatio > 1 ? '@2x' : ''; | 			let hiDPISuffix = window.devicePixelRatio > 1 ? '@2x' : ''; | ||||||
| 			let frag = document.importNode( | 			let frag = document.importNode( | ||||||
| 				MozXULElement.parseXULToFragment(` | 				MozXULElement.parseXULToFragment(` | ||||||
|  | 					<vbox> | ||||||
|  | 						<box anonid="dropmarker-separator"/> | ||||||
|  | 					</vbox> | ||||||
| 					<hbox align="center" anonid="dropmarker-box"> | 					<hbox align="center" anonid="dropmarker-box"> | ||||||
| 						<image src="chrome://zotero/skin/searchbar-dropmarker${hiDPISuffix}.png" width="7" height="4" class="split-menu-button-dropmarker"/> | 						<image src="chrome://zotero/skin/searchbar-dropmarker${hiDPISuffix}.png" width="7" height="4" class="split-menu-button-dropmarker"/> | ||||||
| 					</hbox> | 					</hbox> | ||||||
|  |  | ||||||
|  | @ -721,12 +721,10 @@ | ||||||
| 	overflow: hidden; | 	overflow: hidden; | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| .split-menu-button [anonid="dropmarker-box"]::before { | .split-menu-button [anonid="dropmarker-separator"] { | ||||||
| 	content: ''; | 	-moz-box-flex: 1; | ||||||
| 	display: inline-block; |  | ||||||
| 	height: 12px; |  | ||||||
| 	border-inline-start: 1px solid #aaa; | 	border-inline-start: 1px solid #aaa; | ||||||
| 	margin-inline-end: 2px; | 	margin: 5px 2px 5px 4px; | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| .split-menu-button .split-menu-button-dropmarker { | .split-menu-button .split-menu-button-dropmarker { | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Abe Jellinek
				Abe Jellinek