Improved "thinking" to use bubble and better formatting

jukofyork · Nov 20, 2024 · 3589ed0 · 3589ed0
1 parent 0e2a9db
commit 3589ed0
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 9 deletions.
diff --git a/eclipse.plugin.aiassistant/css/main-style.css b/eclipse.plugin.aiassistant/css/main-style.css
@@ -31,7 +31,6 @@
   --blockquote-text-color: #bbb;
   --blockquote-nested-text-color: #999;
   --blockquote-line-color: #333;
-  --thinking-header-color: #ddd;
   --selected-shadow-blur-radius: 15px;
   --default-margin: 0.5rem;
   --top-margin: 1rem;
@@ -178,14 +177,26 @@ blockquote blockquote {
   margin-left: 0em;
 }
 
+.thinking {
+  background-color: var(--user-bubble-bg-color);
+  border-radius: var(--bubble-radius);
+  padding: var(--small-padding);
+}
+
 .thinking summary {
-  color: var(--thinking-header-color);
+  color: var(--text-color);
+  opacity: 0.8;
+  cursor: pointer;
+  padding: 0 var(--small-padding);
+  margin-left: calc(-1 * var(--default-margin));
 }
 
-/* Adjust spacing inside thinking blocks */
-.thinking blockquote {
-  margin-top: 0.5em;
-  margin-bottom: 0;
+.thinking details {
+  margin-left: var(--default-margin);
+}
+
+.thinking summary:hover {
+  opacity: 1;
 }
 
 /* The default margins are too large */

diff --git a/eclipse.plugin.aiassistant/src/eclipse/plugin/aiassistant/utility/MarkdownParser.java b/eclipse.plugin.aiassistant/src/eclipse/plugin/aiassistant/utility/MarkdownParser.java
@@ -186,7 +186,7 @@ enum BlockType { NONE, CODE, LATEX }
      * @return HTML string containing the opening structure for a thinking block
      */
     private static String getThinkingBlockOpeningHtml() {
-        return "<div class=\"thinking\"><details><summary>Thinking...</summary><blockquote>";
+        return "<div class=\"thinking\"><details><summary>Thinking...</summary>";
     }
 
     /**
@@ -197,7 +197,7 @@ private static String getThinkingBlockOpeningHtml() {
      * @return HTML string containing closing tags for blockquote, details, and div elements
      */
     private static String getThinkingBlockClosingHtml() {
-        return "</blockquote></details></div>";
+        return "</details></div>";
     }
 
     /**
@@ -211,7 +211,7 @@ private static String getThinkingBlockClosingHtml() {
      */
     private static String trimThinkingyBlock(String html) {
         return html
-                .replaceAll("</summary>\\s*<blockquote>(?:\\s|<br/>)+", "</summary><blockquote>")
+                .replaceAll("</summary>(?:\\s|<br/>)+", "</summary><br/>")
                 .replaceAll("</details>\\s*</div>(?:\\s|<br/>)+", "</details></div><br/>");
     }