minor fix: remove ellipsis function

UniversityOfHelsinkiCS · Apr 29, 2024 · 4f2d276 · 4f2d276
1 parent 7fa6632
commit 4f2d276
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 16 deletions.
diff --git a/client/components/ChatBot/Chatbot.scss b/client/components/ChatBot/Chatbot.scss
@@ -36,7 +36,7 @@
     white-space: nowrap;
     overflow: hidden;
     text-overflow: ellipsis;
-    max-width: 30em;
+    max-width: 15em;
     font-size: 1.15rem;
 }
 

diff --git a/client/components/ChatBot/index.js b/client/components/ChatBot/index.js
@@ -13,7 +13,6 @@ import {
     mcExerciseTouched,
 } from 'Utilities/redux/practiceReducer'
 import {
-  cut_and_ellipsis,
   formatGreenFeedbackText,
   hiddenFeatures
 } from 'Utilities/common'
@@ -224,7 +223,7 @@ const Chatbot = () => {
   return (
     <div className="chatbot">
       <Button onClick={toggleCollapse} className="chatbot-toggle"
-              style={{background: "mistyrose"}}>
+              style={{background: "mistyrose", margin: 0}}>
         {isCollapsed ?
          <Icon name="angle up" size='large' /> :
          <Icon name="angle down" size='large' />}
@@ -234,12 +233,9 @@ const Chatbot = () => {
           {/************ current context BANNER = current exercise LEMMA ************/}
           <div className="chatbot-context-info">
             {validToChat && (
-              <div className="chatbot-context-item flex space-between">
-                <div>
-                  {/* make sure the words to display are not too long */
-                    currentWord.choices?.length
-                      ? cut_and_ellipsis(currentWord.choices.join('/'),15)
-                      : currentWord.base}
+              <div className="flex space-between">
+                <div className="chatbot-context-item">
+                  {currentWord.choices?.length ? currentWord.choices.join('/') : currentWord.base}
                 </div>
                 {
                   /* Display lightbulbs */

diff --git a/client/util/common.js b/client/util/common.js
@@ -295,13 +295,6 @@ export const capitalize_first_char_only = word => {
     return p1.toUpperCase() + p2
 }
 
-// if word is too long: cut it off at N characters, attach ellipsis...
-export const cut_and_ellipsis = (word, n) => {
-  if (word.length < n) return word
-
-  const p1 = word.slice(0, n)
-  return p1 + "..."
-}
 
 
 // coloring difficulty