fix some implicit context ui (microsoft#256759)

* fix some implicit context ui

* remove button title

Author: justschen

src/vs/workbench/contrib/chat/browser/attachments/implicitContextAttachment.ts

@@ -75,8 +75,8 @@ export class ImplicitContextAttachmentWidget extends Disposable {
 
 		const uriLabel = this.labelService.getUriLabel(file, { relative: true });
 		const currentFile = localize('openEditor', "Current {0} context", attachmentTypeName);
-		const inactive = localize('enableHint', "disabled");
-		const currentFileHint = currentFile + (this.attachment.enabled || this.attachment.isSelection ? '' : ` (${inactive})`);
+		const inactive = localize('enableHint', "Enable current {0} context", attachmentTypeName);
+		const currentFileHint = this.attachment.enabled || this.attachment.isSelection ? currentFile : inactive;
 		const title = `${currentFileHint}\n${uriLabel}`;
 
 		label.setFile(file, {
@@ -94,7 +94,7 @@ export class ImplicitContextAttachmentWidget extends Disposable {
 
 		if (isSuggestedEnabled) {
 			if (!this.attachment.isSelection) {
-				const buttonMsg = this.attachment.enabled ? localize('disable', "Disable current {0} context", attachmentTypeName) : localize('enable', "Enable current {0} context", attachmentTypeName);
+				const buttonMsg = this.attachment.enabled ? localize('disable', "Disable current {0} context", attachmentTypeName) : '';
 				const toggleButton = this.renderDisposables.add(new Button(this.domNode, { supportIcons: true, title: buttonMsg }));
 				toggleButton.icon = this.attachment.enabled ? Codicon.x : Codicon.plus;
 				this.renderDisposables.add(toggleButton.onDidClick((e) => {

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

@@ -1256,6 +1256,7 @@ have to be updated for changes to the rules above, or to support more deeply nes
 }
 
 .interactive-session .chat-attached-context .chat-attached-context-attachment .monaco-button.codicon.codicon-plus {
+	padding: 0 5px 0 3px;
 	font-size: 12px;
 }
 
@@ -1266,6 +1267,11 @@ have to be updated for changes to the rules above, or to support more deeply nes
 	background: var(--vscode-toolbar-hoverBackground);
 }
 
+.interactive-session .chat-attached-context .chat-attached-context-attachment.implicit.disabled .monaco-button:hover {
+	cursor: pointer;
+	background: transparent;
+}
+
 .interactive-session .chat-attached-context .chat-attached-context-attachment .monaco-icon-label-container {
 	display: flex;