-
Notifications
You must be signed in to change notification settings - Fork 23
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Cody Ignore: Manual autocomplete trigger notification, sidebar comman…
…d button sidebar UX and sundry (#1432) - Intercepts the manual autocomplete trigger and displays a notification - Disables the buttons in the sidebar command tab when in an ignored file - Displays a banner in the sidebar command tab when in an ignored file - No longer displays banners on individual editors - Notification text, tooltip text has been updated to match latest UX proposal https://www.loom.com/share/3a417616ef4f4701ac0e02bcc0c61e64 Part of #1252, #1256 Closes #1253 ## Test plan Manually tested: - Run plugin with environment `CODY_JETBRAINS_FEATURES=cody.feature.internals-menu=true` - Use status bar, Internals, Testing: Cody Ignore and change the policy to match your repo, turn the policy override on. - Check that autocomplete does not happen - Check that the manual autocomplete trigger displays a notification - Check that right click menu actions for Explain, Smell and "Test V1" display a notification; same for their keyboard shortcuts - Check that the buttons in the sidebar command tab are disabled and a banner is displayed - Check that the status bar Cody icon displays a slash and tooltip when in an ignored file - Check that all the above and UX affordances respond when the policy changes (override off, different repo name, etc.)
- Loading branch information
1 parent
99e5835
commit 4dc5732
Showing
10 changed files
with
194 additions
and
103 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
41 changes: 41 additions & 0 deletions
41
src/main/kotlin/com/sourcegraph/cody/ignore/CommandPanelIgnoreBanner.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
package com.sourcegraph.cody.ignore | ||
|
||
import com.intellij.ide.BrowserUtil | ||
import com.intellij.openapi.application.ApplicationManager | ||
import com.intellij.openapi.editor.colors.EditorColors | ||
import com.intellij.openapi.editor.colors.EditorColorsManager | ||
import com.intellij.ui.EditorNotificationPanel | ||
import com.intellij.ui.SideBorder | ||
import com.intellij.ui.components.panels.NonOpaquePanel | ||
import com.sourcegraph.Icons | ||
import com.sourcegraph.common.CodyBundle | ||
import java.awt.Dimension | ||
|
||
class CommandPanelIgnoreBanner() : NonOpaquePanel() { | ||
init { | ||
ApplicationManager.getApplication().assertIsDispatchThread() | ||
|
||
add( | ||
EditorNotificationPanel().apply { | ||
text = CodyBundle.getString("ignore.sidebar-panel-ignored-file.text") | ||
createActionLabel( | ||
CodyBundle.getString("ignore.sidebar-panel-ignored-file.learn-more-cta"), | ||
{ BrowserUtil.browse(CODY_IGNORE_DOCS_URL) }, | ||
false) | ||
icon(Icons.CodyLogoSlash) | ||
}) | ||
|
||
// These colors cribbed from EditorComposite, createTopBottomSideBorder | ||
val scheme = EditorColorsManager.getInstance().globalScheme | ||
val borderColor = | ||
scheme.getColor(EditorColors.SEPARATOR_ABOVE_COLOR) | ||
?: scheme.getColor(EditorColors.TEARLINE_COLOR) | ||
border = SideBorder(borderColor, SideBorder.TOP or SideBorder.BOTTOM) | ||
} | ||
|
||
override fun getMaximumSize(): Dimension { | ||
val size = super.getMaximumSize() | ||
size.height = preferredSize.height | ||
return size | ||
} | ||
} |
55 changes: 0 additions & 55 deletions
55
src/main/kotlin/com/sourcegraph/cody/ignore/IgnoreNotificationProvider.kt
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.