SDU-eScience / UCloud

https://docs.cloud.sdu.dk
European Union Public License 1.2
19 stars 7 forks source link

(More pt. 2 pt II) UI-redesign issues #4141

Closed Tiggles closed 5 months ago

Tiggles commented 7 months ago

Essential

In the middle

Minor

Tiggles commented 7 months ago

4095 Previous

tedyted26 commented 7 months ago
Screenshot 2024-03-04 at 11 02 37

Creating a folder from the add folder menu when configuring a job, then clicking "use" leaves this on the job parameters

tedyted26 commented 7 months ago

Clicking on "Manage syncronization" from modal in any job parameter, and then going back, resets all my job parameters

tedyted26 commented 7 months ago

"here" link in "Create ssh" screen has the same color as the rest of the text, even though it's a link

Tiggles commented 7 months ago
Screenshot 2024-03-04 at 15 08 05
xiroV commented 7 months ago

Empty trash modal needs restyling. Could be resized, and doesn't add the background overlay. Also I don't think we use the underline of modal-headings anymore.

Empty-ing of trash redirects to /drives page. Would expect either no redirect, or redirect to the root of the drive that the trash folder belongs to.

DanThrane commented 6 months ago
image
Tiggles commented 6 months ago

image

Should have been fixed in d4f2dfd149865521bb61263cea1b0b89282c1a86

tedyted26 commented 6 months ago

(Not sure if its an issue or a design choice) In job parameters, when using the buttons to add hours, if the current selected hours are less than the hours the button adds, the number does not add up, but changes to the one on the button. For example, having 1h and pressing +8 will result in 8h, not 9. Having 8h and pressing +24 will result in 24h, not 32.

Tiggles commented 6 months ago

It's a design choice, but I'm not sure we're done tuning it yet.

tedyted26 commented 6 months ago
Screenshot 2024-03-11 at 10 54 40

Shape (shadow?) of hold to confirm icon, and color of the done icon after holding (its red on a red background)

DanThrane commented 6 months ago

It is not possible to create a folder when selecting an input folder for an app. The enter key does nothing when entering the name.

DanThrane commented 6 months ago
image

Open terminal and open interface are placed incorrectly with multi-node jobs (I feel like you have mentioned a fix already, but just in case).

Tiggles commented 6 months ago

It is not possible to create a folder when selecting an input folder for an app. The enter key does nothing when entering the name.

Already fixed

Tiggles commented 6 months ago

Job selector styling is off for column titles

Screenshot 2024-03-13 at 09 54 59
MHvegholm commented 6 months ago

image Font color difficult to read in some dropdown menus:

MHvegholm commented 6 months ago

image Workspace > usage leads to error screen, cannot escape without reloading (Crash?) – Only in my workspace

Tiggles commented 6 months ago

Workspace > usage leads to error screen, cannot escape without reloading (Crash?) – Only in my workspace

* When clicking on “Workspace” icon or accessing “workspace/usage” page while within “My workspace”, leads to error report page which cannot be left without reloading website ( Crash?)

* Error doesn’t occur when in workspace “Provider Type 1”

I'm not experiencing this on sandbox.dev.cloud.sdu.dk, so I'd like to check your console output the next time you are here. Tested it on MacOS with Firefox, Safari, and Chrome

Edit: this seems to be related to the ucloud-msg format

MHvegholm commented 6 months ago

image

Suggestion: addition of “jump-to workspace Rootdir/drive selection menu"

MHvegholm commented 6 months ago

image Suggestion: Embiggen “star” icon for “starring apps”

Tiggles commented 6 months ago

Favorite icon flicker on file browse component. Maybe related to :hover rule being temporarily disabled? https://github.com/SDU-eScience/UCloud/assets/7610257/77635ad4-37dd-40fa-b6f8-e6ad5b278eed

DanThrane commented 6 months ago

It should not be possible to select text in the flavor dropdown:

image

Missing placeholder text for license servers:

image

Placeholder text is invisible on login page (compare this with production):

image
Tiggles commented 6 months ago

https://github.com/SDU-eScience/UCloud/assets/7610257/8872097f-f04f-4035-aa36-22f4bf76ddb8

DanThrane commented 6 months ago

Surprising behavior during drive creation:

  1. Click 'Create drive'
  2. Click somewhere else on the screen such that the input loses focus
  3. This triggers a creation event which fails due to the title being blank

Generally this is the correct behavior, but I think we should check if the input field is empty. If the input field is empty then we should consider this a cancel instead of a create. If the input field is not empty, then we should continue treating it as a creation request.

DanThrane commented 6 months ago
image

Search query is not empty.

JHavstein commented 6 months ago

In the 'Files' menu, the buttons Upload files and Create folder have keyboard shortcuts shown in the button (picture 1). These keyboard shortcuts are missing from the same two buttons from the job output page (picture 2). The keyboard shortcuts work in both places.

Skærmbillede 2024-03-27 kl  13 25 51 Skærmbillede 2024-03-27 kl  13 26 09
DanThrane commented 6 months ago

"Launch with..." fails with a 400 bad request from the backend. This is a frontend issue since the wrong request type is being sent. There might still be backend issues once we begin sending the correct request type.

image
Tiggles commented 6 months ago

https://github.com/SDU-eScience/UCloud/assets/7610257/102e5095-5af1-4b09-95db-a670b2befed6

DanThrane commented 6 months ago

I think we should add an edge-case for jobs running with the unknown application (name = unknown). In this case it should probably skip the application name entirely and just display job name/id.

image
Tiggles commented 6 months ago

Weirdly, this is partly handled already:

{app.name === "unknown" ? null :
            <Link to={buildQueryString(`/jobs/create`, {app: app.name, version: app.version, import: job.id})}>
                <Button>Run application again</Button>
            </Link>
        }

I think we should add an edge-case for jobs running with the unknown application (name = unknown). In this case it should probably skip the application name entirely and just display job name/id.

image
Tiggles commented 5 months ago

Not wide enough width for selector (on Chrome) image

DanThrane commented 5 months ago

Drive rename field overlaps with provider icon

image
Tiggles commented 5 months ago

Delete Drive lost it's color

Screenshot 2024-04-08 at 10 32 15
Tiggles commented 5 months ago

Drive-dropdown in FileBrowse is too wide

Screenshot 2024-04-09 at 11 57 38
Tiggles commented 5 months ago

Long breadcrumbs can push context switcher Ellipses are added, even when no folder is present. (Only in one folder in the drive, in the image)

Screenshot 2024-04-11 at 13 51 09
mrlhansen commented 5 months ago

image image

xiroV commented 5 months ago

A few comments (current dev state):

DanThrane commented 5 months ago

Markdown preview links are no longer correct due to changes in preview URL.

Tiggles commented 5 months ago

Markdown preview links are no longer correct due to changes in preview URL.

I think I need some more details

Screenshot 2024-04-16 at 09 34 27
DanThrane commented 5 months ago

Sizing of console is slightly wrong (low priority in my opnion)

image

mrlhansen commented 5 months ago

The issue with URLs ending in a / also causes the link to files to not work here (also, should that really open in a new window?). image

Tiggles commented 5 months ago

(also, should that really open in a new window?)

Yes, otherwise the form/arguments are wiped when returning to the page.

DanThrane commented 5 months ago

Browsing files causes GET /api/jobs/retrieveProducts?providers=$PROVIDER_ID and GET /api/iapps/syncthing/retrieve?provider=$PROVIDER_ID&productId=syncthing to be invoked on every folder instead of being cached.

Browsing any of the resources pages causes 3x the same browse request (e.g. GET /api/licenses/browse?itemsPerPage=100&status=READY). Note that all requests send the same parameter, so it is not trying to prefetch multiple pages in this case.

mrlhansen commented 5 months ago

On the SSH keys page (https://dev.cloud.sdu.dk/app/ssh-keys).

DanThrane commented 5 months ago

Drag-to-select no longer works if you are starting the drag from the "outside". That is not on a row. Similarly, it often doesn't work when dragging to the outside.

Tiggles commented 5 months ago

Did it work before, dragging from the outside in?

Tiggles commented 5 months ago

Next #4177