Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delete workspace gui issues #5997

Closed
hm2075 opened this issue Oct 3, 2021 · 1 comment · Fixed by #6038
Closed

Delete workspace gui issues #5997

hm2075 opened this issue Oct 3, 2021 · 1 comment · Fixed by #6038
Labels

Comments

@hm2075
Copy link

hm2075 commented Oct 3, 2021

Bug description

image

long project names do not fit in window

Steps to reproduce

delete workspace that has a long project name

Expected behavior

No response

Example repository

No response

Anything else?

No response

@gtsiolis
Copy link
Contributor

gtsiolis commented Oct 4, 2021

Nice catch @hm2075! We recently fixed this issue for a similar modal in #5248 (see also #4556).

The fix should be relatively simple if you're interested in contributing with a fix. Something like the followign could suffice. 🏓

diff --git a/components/dashboard/src/components/ConfirmationModal.tsx b/components/dashboard/src/components/ConfirmationModal.tsx
index e7f9327e..20c6c562 100644
--- a/components/dashboard/src/components/ConfirmationModal.tsx
+++ b/components/dashboard/src/components/ConfirmationModal.tsx
@@ -33,7 +33,7 @@ export default function ConfirmationModal(props: {
             child.push(
                 <div className="w-full p-4 mb-2 bg-gray-100 dark:bg-gray-700 rounded-xl group">
                     <p className="text-base text-gray-800 dark:text-gray-100 font-semibold">{props.children.name}</p>
-                    {props.children.description && <p className="text-gray-500">{props.children.description}</p>}
+                    {props.children.description && <p className="text-gray-500 truncate">{props.children.description}</p>}
                 </div>
             )
         } else if (Array.isArray(props.children)) {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants