Arvados

22208-project-details @ 6eb4ecd87273301ba12df451a0ab7ca23d6923af

developer-run-tests: #4512

22208-project-details @ 0a08e5e5a09f95983e9f6e20d15e3566b697f48f

developer-run-tests: #4513

22208-project-details @ 0abf4f0320a39721a400eedc2dba29e1ee19ec57

developer-run-tests: #4515

• All agreed upon points are implemented / addressed. Describe changes from pre-implementation design.
  • makes the design changes that were discussed at meeting Oct 16 meeting with SWZ
• Anything not implemented (discovered or discussed during work) has a follow-up story.
  • n/a
• Code is tested and passing, both automated and manual, what manual testing was done is described.
  • yes
• New or changed UX/UX and has gotten feedback from stakeholders.
  • changes are based on feedback
• Documentation has been updated.
  • n/a
• Behaves appropriately at the intended scale (describe intended scale).
  • no changes to scale
• Considered backwards and forwards compatibility issues between client and server.
  • no changes to API
• Follows our coding standards and GUI style guidelines.
  • yes

project-details-card.tsx L119 should be deleted

The feedback we got from Roche when we first showed them the Details Card was that they wanted the first line of the description to be shown before expanding. I know we are going in a different direction, but be aware that they will likely bring that up again.

I'm not in love with the toggle being a chevron to the left of the project title. Functionally it's fine, but aesthetically it sort of clunkifies things. The trade-off is that we get a single-line details card, which does like nice. I don't think there is a great solution for this either, although a simple "show description / hide description" button to the right of the project title might work, it's hard to say.

22208-project-details @ 47757da958289b062ad0aba39bd0e1bdf6cf0da6

developer-run-tests: #4518

LGTM