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

Redesign right-hand info sidebar #7110

Merged
merged 33 commits into from
Jun 21, 2023
Merged

Redesign right-hand info sidebar #7110

merged 33 commits into from
Jun 21, 2023

Conversation

hotzenklotz
Copy link
Member

@hotzenklotz hotzenklotz commented May 31, 2023

PR slightly redesigns the right-hand info sidebar to be more in line with the new branding and design language.

Changes:

  • redesigned right-hand side bar info tab for View mode and Annotation mode
  • moved the AI Segmentation modals to the "main" annotation menu
    • refined wording a bit
    • renamed the modals to "AI Segmentation". The word inferral is very techy.
  • added new icons
image image

URL of deployed dev instance (used for testing):

Steps to test:

  • Open any DS in "view" and "annotation" modes
  • Share them with yourself and try view as anonymous user
  • Add descriptions and names to DS and annotations
  • Try in dark mode

Issues:


(Please delete unneeded items, merge only when none are left open)

@hotzenklotz hotzenklotz self-assigned this May 31, 2023
…esign-right-sidebar

* 'master' of github.com:scalableminds/webknossos:
  When adding annotation layer, assert no duplicate names (#7103)
  Add folders to dataset table (#6996)
…esign-right-sidebar

* 'master' of github.com:scalableminds/webknossos:
  Create bounding box by dragging with box tool (#7118)
  Prevent 'negative' buckets from being created (#7124)
  Lazy load onnx and canvas2html module (#7121)
  Disable editing of super voxel skeletons in skeleton mode (#7086)
  add missing evolution to migration guide (#7126)
  Change sttp backend to HttpURLConnectionBackend (#7125)
  Implement Zarr v3 and sharding codec (#7079)
  Fix decompression of garbage data after valid gzip data causing decompression to fail (#7119)
  When scanning volume buckets, skip those with unparseable key (#7115)
@hotzenklotz hotzenklotz changed the title WIP Redesign right-hand info sidebar Redesign right-hand info sidebar Jun 8, 2023
Copy link
Member

@philippotto philippotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superb stuff 👍 This looks like a very big design increment 🥇 Will test now.

public/images/icon-mouse-left.svg Outdated Show resolved Hide resolved
</tbody>
</table>
{this.renderSelectedStartingJobsModal()}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so, these actions can only be triggered from an annotation (instead of from the dataset view) now, right?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. That was not intentional. I think this menu entry should be available in both situations. Will fix.

@philippotto
Copy link
Member

philippotto commented Jun 8, 2023

My findings from testing:

  • 1.1) cog icon goes to new line when DS name is too long
  • 1.2 + 1.3) text color is too light in comparison to dataset description and shortcut intro. in the original design, this was more consistent I think.

  • 2.1) the shortcuts "tags" are too bright in dark mode. maybe this is automatically better when the text gets brighter, too. if not, I'd tone it done a bit with sth like opacity: 0.85.

  • 3.1) the edit-icon is over the text if the text has an unlucky length.
  • 3.2) the margin betwen statistics-headline and statistics-content is too large (at least when compared with the margin above the stats-headline).

  • 4.1 + 4.2) the line thickness of these icons seems to be narrower when compared to the other icons. this makes them a bit harder to see in my opinion.

hotzenklotz and others added 8 commits June 9, 2023 11:49
…esign-right-sidebar

* 'master' of github.com:scalableminds/webknossos:
  Fix true brush size for small, uneven brush sizes (#7129)
…b_modals.tsx

Co-authored-by: Philipp Otto <philippotto@users.noreply.github.com>
…b_modals.tsx

Co-authored-by: Philipp Otto <philippotto@users.noreply.github.com>
…knossos into redesign-right-sidebar

* 'redesign-right-sidebar' of github.com:scalableminds/webknossos:
  Update frontend/javascripts/oxalis/view/right-border-tabs/starting_job_modals.tsx
  Update frontend/javascripts/oxalis/view/right-border-tabs/starting_job_modals.tsx
…o_tab_view.tsx

Co-authored-by: Philipp Otto <philippotto@users.noreply.github.com>
…knossos into redesign-right-sidebar

* 'redesign-right-sidebar' of github.com:scalableminds/webknossos:
  Update frontend/javascripts/oxalis/view/right-border-tabs/dataset_info_tab_view.tsx
…esign-right-sidebar

* 'master' of github.com:scalableminds/webknossos:
  Layer actions overflow menu (#7123)
  Unify DatasetPath and VaultPath, cleaner ChunkReader creation (#7134)
  Segment to Bucket Index for Volume Annotations (#7063)
  Reverse compression on sharded neuroglancer datasets (#7135)
…esign-right-sidebar

* 'master' of github.com:scalableminds/webknossos:
  quick fixes for overflow menu (#7138)
@hotzenklotz hotzenklotz marked this pull request as ready for review June 14, 2023 07:20
…esign-right-sidebar

* 'master' of github.com:scalableminds/webknossos:
  added Youtube videos to docs
  Log dataset uploads (with no conversion) to slack (#7157)
  Added "Automation Tutorial" to docs (#7160)
  fix logo image in README.md
  Second try for “Async IO for HttpsDataVault, Fox Error Handling” (#7155)
  Revert "Async IO for HttpsDataVault, Fox Error Handling (#7137)" (#7154)
  Async IO for HttpsDataVault, Fox Error Handling (#7137)
  Fix vault path for precomputed datasets (#7151)
  Add extended keyboard shortcut mode via ctrl + k for tool shortcuts (#7112)
  Shared Chunk Cache for all DatasetArrays, CacheWeight for AlfuCache (#7067)
@hotzenklotz hotzenklotz enabled auto-merge (squash) June 21, 2023 05:55
@hotzenklotz
Copy link
Member Author

@philippotto Anymore review changes or is this free to merge?

Copy link
Member

@philippotto philippotto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢 🚢 🚢

@hotzenklotz hotzenklotz merged commit be790b7 into master Jun 21, 2023
2 checks passed
@hotzenklotz hotzenklotz deleted the redesign-right-sidebar branch June 21, 2023 08:25
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 this pull request may close these issues.

Redesign info tab in right sidebar
3 participants