Project

General

Profile

Actions

Bug #17264

closed

Files in file picker have wrong icon (folder instead of file)

Added by Peter Amstutz almost 4 years ago. Updated 8 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench2
Target version:
Story points:
-
Release relationship:
Auto

Subtasks 1 (0 open1 closed)

Task #17265: ReviewClosedDaniel Kutyła01/21/2021Actions
Actions #1

Updated by Peter Amstutz almost 4 years ago

  • Assigned To set to Daniel Kutyła
Actions #2

Updated by Peter Amstutz almost 4 years ago

  • Target version changed from 2021-01-20 Sprint to 2021-02-03 Sprint
Actions #3

Updated by Daniel Kutyła almost 4 years ago

  • Status changed from New to In Progress
Actions #4

Updated by Daniel Kutyła almost 4 years ago

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/534df712c4cfc3f6574dee1f4906bb7f6cc70be9
Test run: developer-tests-workbench2: #2 ​49/

This is the simplest yet the fastest way to implement this, I would vote for a cleanup ticket for the tree components all over the app as this is really messy.

Actions #5

Updated by Peter Amstutz almost 4 years ago

I noticed that CollectionIcon and FileIcon use the same "LibraryBooks" material ui icon. Let's change what FileIcon uses. I think the one from the material UI icons that works best is the Description icon. Could you make this change?

workbench2/src/components/icon/icon.tsx

export const FileIcon: IconType = (props) => <Description {...props} />;

LGTM with that tweak.

Actions #6

Updated by Daniel Kutyła almost 4 years ago

  • % Done changed from 0 to 100
  • Status changed from In Progress to Resolved
Actions #7

Updated by Peter Amstutz 8 months ago

  • Release set to 37
Actions

Also available in: Atom PDF