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

Toggling the Inspector in Media View doesn't consistently resize the thumbnail grid. #2427

Open
0xBA5E64 opened this issue Apr 30, 2024 · 0 comments
Labels
kind/bug Something isn't working status/needs-triage

Comments

@0xBA5E64
Copy link
Contributor

Describe the bug

When toggling the Inspector sidebar in Spacedrive, the media view thumbnail grid won't always consistently resize to make space on the side, instead displaying underneath the Inspector itself. Similarly, as the Inspector is closed, the thumbnail grid won't consistently re-claim said space:

Kooha-2024-04-30-20-37-57-.00.00.000-00.51.333.mp4

This is consistent across all platforms I've tried (see Platform and versions below).

This seems to fix itself as the user scrolls down in the grid (likely causing some sort of update), but as you're likely not doing this while toggling the Inspector, this is quite apparent.

Reproduction

  1. Open an indexed Location
  2. Switch to Media View in the top bar or via Ctrl+3
  3. Toggle the Inspector from the top right or by pressing Ctrl+I, repeatedly.
  4. Observe as the Media View thumbnails randomly "clip" underneath the Inspector, or occasionally refuses to re-fill the space used by the Inspector

Expected behavior

The Media View thumbnails should only be compacted as the Inspector is displayed, and go back to using the full screen width once disabled.

Platform and versions

This has been tested on:
 - macOS Sonoma (v0.2.13)
 - Docker-container Web UI (`staging` tag: v0.2.12 - 2b5e03e)
 - Arch Linux (Unofficial "spacedrive" AUR-build @ v0.2.11 - eb16568) [current]
  - `pnpm 9.0.5 / cargo 1.77.1 (e52e36006 2024-03-26) / rustc 1.77.1 (7cf61ebde 2024-03-27)`

Stack trace

N/A

Additional context

The recording may seem laggy due to the sub-par performance I'm currently experiencing with the Linux build of the app. This may in part be due to the unofficial AUR build, but having tried the officially appimage build previously it was not much better unfortunately. (Performance in the browser is great however, yet experiences the same buggyness)

@0xBA5E64 0xBA5E64 added kind/bug Something isn't working status/needs-triage labels Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working status/needs-triage
Projects
None yet
Development

No branches or pull requests

1 participant