fiftyone icon indicating copy to clipboard operation
fiftyone copied to clipboard

Published 20 hours ago • verified publisher icon voxel51

[FR] Expanded View, shading covers labels (on top-left)

Open oguz-hanoglu opened this issue 3 years ago • 1 comments

Proposal Summary

While navigating samples in expanded view, the labels are shown on top-left, which is nice. However, when the mouse is on the image, they are shaded. The top-most one gets almost invisible.

Motivation

  • What is the use case for this feature?

Navigating samples in expanded view while easily reading their labels

  • Why is this use case valuable to support for FiftyOne users in general?

Seeing the labels is a highly valuable necessity when using the GUI.

  • Why is this use case valuable to support for your project(s) or organization?

I will be able to follow the sample labels while navigating my samples in Fiftyone.

  • Why is it currently difficult to achieve this use case? (please be as specific as possible about why related FiftyOne features and components are insufficient)

I cannot see what is behind when the label is shaded with a black strip.

What areas of FiftyOne does this feature affect?

  • [x] App: FiftyOne application
  • [ ] Core: Core fiftyone Python library
  • [ ] Server: FiftyOne server

Details

  • Some minor UI tweaks are likely to solve this problem (according to @ehofesmann)

Willingness to contribute

The FiftyOne Community encourages new feature contributions. Would you or another member of your organization be willing to contribute an implementation of this feature?

  • [ ] Yes. I can contribute this feature independently.
  • [ ] Yes. I would be willing to contribute this feature with guidance from the FiftyOne community.
  • [x] No. I cannot contribute this feature at this time.

oguz-hanoglu avatar Oct 14 '22 13:10 oguz-hanoglu

https://user-images.githubusercontent.com/99868596/196111695-6e42bb70-683f-4887-b176-c1598ddc4ee8.mp4

oguz-hanoglu avatar Oct 17 '22 07:10 oguz-hanoglu

Hi @oguz-hanoglu. Note that you can zoom out so the label(s) are not covered.

benjaminpkane avatar Oct 27 '22 19:10 benjaminpkane

Hi @benjaminpkane, zoom out is a valid but not practical solution. My use case is navigating the samples, for that I constantly press right arrow and analyze the outputs. To activate the right arrow, cursor needs to be on the image and this triggers shadow.

oguz-hanoglu avatar Oct 28 '22 05:10 oguz-hanoglu

As a solution, zoom can persist when moved to next sample by right arrow.

oguz-hanoglu avatar Oct 28 '22 05:10 oguz-hanoglu

As a solution, zoom can persist when moved to next sample by right arrow.

It's a good idea. I will investigate

benjaminpkane avatar Oct 28 '22 17:10 benjaminpkane

This is still very much needed. I believe it could greatly improve the user experience. Shading gets annoying when you have important data at the edges and zooming is cumbersome when you like to investigate changes in a specific region of interest.

I understand that the team might be occupied with other tasks, but I just wanted to emphasize the continued importance of this request. I'm looking forward to seeing this feature being added in a future release. Thank you very much for your consideration.

oguz-hanoglu avatar Jul 04 '23 08:07 oguz-hanoglu

Hi @oguz-hanoglu. It is a good feature to add. This would likely not be the default behavior because aspect ratio and resolution can vary greatly between samples, and patches views, etc. can zoom to label content content so there are some logical details to consider. Along with debouncing zoom/pan events before persisting. In some free time, I will try prototyping it and let you know 👍

benjaminpkane avatar Jul 05 '23 16:07 benjaminpkane