web - firefox fit-content is -moz-fit-content

Benjamin Pasero · Benjamin Pasero · commit 3782d658f89d · 2019-10-25T15:55:57.000+02:00
diff --git a/src/vs/base/browser/ui/dialog/dialog.css b/src/vs/base/browser/ui/dialog/dialog.css
@@ -178,6 +178,7 @@
 
 .monaco-workbench .dialog-box > .dialog-buttons-row > .dialog-buttons > .monaco-button {
 	max-width: fit-content;
+	max-width: -moz-fit-content;
 	padding: 5px 10px;
 	margin: 4px 5px; /* allows button focus outline to be visible */
 	overflow: hidden;
diff --git a/src/vs/base/browser/ui/list/listView.ts b/src/vs/base/browser/ui/list/listView.ts
@@ -20,6 +20,7 @@ import { Range, IRange } from 'vs/base/common/range';
 import { equals, distinct } from 'vs/base/common/arrays';
 import { DataTransfers, StaticDND, IDragAndDropData } from 'vs/base/browser/dnd';
 import { disposableTimeout, Delayer } from 'vs/base/common/async';
+import { isFirefox } from 'vs/base/browser/browser';
 
 interface IItem<T> {
 	readonly id: string;
@@ -595,7 +596,7 @@ export class ListView<T> implements ISpliceable<T>, IDisposable {
 			return;
 		}
 
-		item.row.domNode.style.width = 'fit-content';
+		item.row.domNode.style.width = isFirefox ? '-moz-fit-content' : 'fit-content';
 		item.width = DOM.getContentWidth(item.row.domNode);
 		const style = window.getComputedStyle(item.row.domNode);
 
diff --git a/src/vs/base/parts/tree/browser/treeView.ts b/src/vs/base/parts/tree/browser/treeView.ts
@@ -265,7 +265,7 @@ export class ViewItem implements IViewItem {
 			}
 
 			if (this.context.horizontalScrolling) {
-				this.element.style.width = 'fit-content';
+				this.element.style.width = Browser.isFirefox ? '-moz-fit-content' : 'fit-content';
 			}
 
 			try {
@@ -289,7 +289,7 @@ export class ViewItem implements IViewItem {
 
 		const style = window.getComputedStyle(this.element);
 		const paddingLeft = parseFloat(style.paddingLeft!);
-		this.element.style.width = 'fit-content';
+		this.element.style.width = Browser.isFirefox ? '-moz-fit-content' : 'fit-content';
 		this.width = DOM.getContentWidth(this.element) + paddingLeft;
 		this.element.style.width = '';
 	}
diff --git a/src/vs/workbench/browser/parts/editor/media/editorstatus.css b/src/vs/workbench/browser/parts/editor/media/editorstatus.css
@@ -48,4 +48,5 @@
 	padding-right: 12px;
 	margin-right: 5px;
 	max-width: fit-content;
+	max-width: -moz-fit-content;
 }
diff --git a/src/vs/workbench/browser/parts/editor/media/tabstitlecontrol.css b/src/vs/workbench/browser/parts/editor/media/tabstitlecontrol.css
@@ -53,13 +53,15 @@
 .monaco-workbench .part.editor > .content .editor-group-container > .title .tabs-container > .tab.sizing-fit {
 	width: 120px;
 	min-width: fit-content;
+	min-width: -moz-fit-content;
 }
 
 .monaco-workbench .part.editor > .content .editor-group-container > .title .tabs-container > .tab.sizing-shrink {
 	min-width: 60px;
 	flex-basis: 0; /* all tabs are even */
 	flex-grow: 1; /* all tabs grow even */
 	max-width: fit-content;
+	max-width: -moz-fit-content;
 }
 
 .monaco-workbench .part.editor > .content .editor-group-container > .title .tabs-container > .tab.sizing-shrink.close-button-left::after,
diff --git a/src/vs/workbench/browser/parts/notifications/media/notificationsList.css b/src/vs/workbench/browser/parts/notifications/media/notificationsList.css
@@ -107,6 +107,7 @@
 
 .monaco-workbench .notifications-list-container .notification-list-item .notification-list-item-buttons-container .monaco-button {
 	max-width: fit-content;
+	max-width: -moz-fit-content;
 	padding: 5px 10px;
 	margin: 4px 5px; /* allows button focus outline to be visible */
 	font-size: 12px;
diff --git a/src/vs/workbench/contrib/debug/browser/media/debugViewlet.css b/src/vs/workbench/contrib/debug/browser/media/debugViewlet.css
@@ -224,6 +224,7 @@
 	flex: 1;
 	flex-shrink: 0;
 	min-width: fit-content;
+	min-width: -moz-fit-content;
 }
 
 .debug-viewlet .debug-call-stack .stack-frame.subtle {
diff --git a/src/vs/workbench/contrib/extensions/browser/media/extensionsViewlet.css b/src/vs/workbench/contrib/extensions/browser/media/extensionsViewlet.css
@@ -191,6 +191,7 @@
 	font-size: 80%;
 	padding-left: 6px;
 	min-width: fit-content;
+	min-width: -moz-fit-content;
 }
 
 .extensions-viewlet:not(.narrow) > .extensions .extension > .details > .header-container > .header > .version {
diff --git a/src/vs/workbench/contrib/files/browser/media/explorerviewlet.css b/src/vs/workbench/contrib/files/browser/media/explorerviewlet.css
@@ -63,6 +63,7 @@
 
 .explorer-viewlet .panel-header .count {
 	min-width: fit-content;
+	min-width: -moz-fit-content;
 	display: flex;
 	align-items: center;
 }

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ export class ViewItem implements IViewItem {`
`265`	`265`	`}`
`266`	`266`
`267`	`267`	`if (this.context.horizontalScrolling) {`
`268`		`- this.element.style.width = 'fit-content';`
	`268`	`+ this.element.style.width = Browser.isFirefox ? '-moz-fit-content' : 'fit-content';`
`269`	`269`	`}`
`270`	`270`
`271`	`271`	`try {`
`@@ -289,7 +289,7 @@ export class ViewItem implements IViewItem {`
`289`	`289`
`290`	`290`	`const style = window.getComputedStyle(this.element);`
`291`	`291`	`const paddingLeft = parseFloat(style.paddingLeft!);`
`292`		`- this.element.style.width = 'fit-content';`
	`292`	`+ this.element.style.width = Browser.isFirefox ? '-moz-fit-content' : 'fit-content';`
`293`	`293`	`this.width = DOM.getContentWidth(this.element) + paddingLeft;`
`294`	`294`	`this.element.style.width = '';`
`295`	`295`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,4 +48,5 @@`
`48`	`48`	`padding-right: 12px;`
`49`	`49`	`margin-right: 5px;`
`50`	`50`	`max-width: fit-content;`
	`51`	`+ max-width: -moz-fit-content;`
`51`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -53,13 +53,15 @@`
`53`	`53`	`.monaco-workbench .part.editor > .content .editor-group-container > .title .tabs-container > .tab.sizing-fit {`
`54`	`54`	`width: 120px;`
`55`	`55`	`min-width: fit-content;`
	`56`	`+ min-width: -moz-fit-content;`
`56`	`57`	`}`
`57`	`58`
`58`	`59`	`.monaco-workbench .part.editor > .content .editor-group-container > .title .tabs-container > .tab.sizing-shrink {`
`59`	`60`	`min-width: 60px;`
`60`	`61`	`flex-basis: 0; /* all tabs are even */`
`61`	`62`	`flex-grow: 1; /* all tabs grow even */`
`62`	`63`	`max-width: fit-content;`
	`64`	`+ max-width: -moz-fit-content;`
`63`	`65`	`}`
`64`	`66`
`65`	`67`	`.monaco-workbench .part.editor > .content .editor-group-container > .title .tabs-container > .tab.sizing-shrink.close-button-left::after,`
Original file line number	Diff line number	Diff line change
`@@ -224,6 +224,7 @@`
`224`	`224`	`flex: 1;`
`225`	`225`	`flex-shrink: 0;`
`226`	`226`	`min-width: fit-content;`
	`227`	`+ min-width: -moz-fit-content;`
`227`	`228`	`}`
`228`	`229`
`229`	`230`	`.debug-viewlet .debug-call-stack .stack-frame.subtle {`
Original file line number	Diff line number	Diff line change
`@@ -191,6 +191,7 @@`
`191`	`191`	`font-size: 80%;`
`192`	`192`	`padding-left: 6px;`
`193`	`193`	`min-width: fit-content;`
	`194`	`+ min-width: -moz-fit-content;`
`194`	`195`	`}`
`195`	`196`
`196`	`197`	`.extensions-viewlet:not(.narrow) > .extensions .extension > .details > .header-container > .header > .version {`
Original file line number	Diff line number	Diff line change
`@@ -63,6 +63,7 @@`
`63`	`63`
`64`	`64`	`.explorer-viewlet .panel-header .count {`
`65`	`65`	`min-width: fit-content;`
	`66`	`+ min-width: -moz-fit-content;`
`66`	`67`	`display: flex;`
`67`	`68`	`align-items: center;`
`68`	`69`	`}`