fix some chat ui again (microsoft#250032)

* fix some chat ui again

* fix display on render

Author: justschen

src/vs/workbench/contrib/chat/browser/chatAttachmentWidgets.ts

@@ -326,14 +326,13 @@ function createImageElements(resource: URI | undefined, name: string, fullName:
 	const hoverElement = dom.$('div.chat-attached-context-hover');
 	hoverElement.setAttribute('aria-label', ariaLabel);
 
-	if (!supportsVision && currentLanguageModel) {
+	if ((!supportsVision && currentLanguageModel) || omittedState === OmittedState.Full) {
 		element.classList.add('warning');
 		hoverElement.textContent = localize('chat.fileAttachmentHover', "{0} does not support this {1} type.", currentLanguageModelName, 'image');
 		disposable.add(hoverService.setupDelayedHover(element, { content: hoverElement, appearance: { showPointer: true } }));
 	} else {
 		disposable.add(hoverService.setupDelayedHover(element, { content: hoverElement, appearance: { showPointer: true } }));
 
-
 		const blob = new Blob([buffer as Uint8Array<ArrayBuffer>], { type: 'image/png' });
 		const url = URL.createObjectURL(blob);
 		const pillImg = dom.$('img.chat-attached-context-pill-image', { src: url, alt: '' });

src/vs/workbench/contrib/chat/browser/media/chat.css

@@ -1869,6 +1869,10 @@ have to be updated for changes to the rules above, or to support more deeply nes
 
 .interactive-session:not(.chat-widget > .interactive-session) {
 
+	.interactive-item-container {
+		padding-top: 5px 16px;
+	}
+
 	.interactive-item-container.interactive-request {
 		align-items: flex-end;
 		padding-bottom: 0px;
@@ -1883,7 +1887,6 @@ have to be updated for changes to the rules above, or to support more deeply nes
 		margin-left: auto;
 		width: fit-content;
 		margin-bottom: 5px;
-		box-shadow: 0 1px 2px rgba(0, 0, 0, 0.1);
 		position: relative;
 	}
 
@@ -1901,7 +1904,6 @@ have to be updated for changes to the rules above, or to support more deeply nes
 	}
 
 	.interactive-item-container.interactive-request .chat-attached-context {
-		margin-top: 10px;
 		max-width: 100%;
 		width: fit-content;
 		justify-content: flex-end;
@@ -1940,6 +1942,10 @@ have to be updated for changes to the rules above, or to support more deeply nes
 		padding-top: 4px;
 		padding-left: 1px;
 	}
+
+	.request-hover.has-no-actions {
+		display: none !important;
+	}
 }
 
 .chat-buttons-container {