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 · 2 comments
Labels
app/desktop app/web Bug Created by Linear-GitHub Sync Desktop (Client) Created by Linear-GitHub Sync good first issue Good for newcomers High priority Created by Linear-GitHub Sync kind/bug Something isn't working web

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
@HeavenVolkoff HeavenVolkoff added good first issue Good for newcomers app/desktop app/web web Bug Created by Linear-GitHub Sync High priority Created by Linear-GitHub Sync Desktop (Client) Created by Linear-GitHub Sync and removed status/needs-triage labels Jun 4, 2024
@HeavenVolkoff
Copy link
Member

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:

Oh, this is certainly not working as it should. I'll raise this internally, and we will fix it as soon as possible

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)

Yeah, this is a known issue, unfortunately Linux releases are currently locked to gtk3, which uses an old webkit2gtk version that has some performance issues with complex web apps. This is due to Tauri not supporting gtk4, we are currently looking for an alternative solution specifically Linux

@0xBA5E64
Copy link
Contributor Author

0xBA5E64 commented Jun 6, 2024

I can confirm this issue still persists on version 0.3.1, running on macOS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app/desktop app/web Bug Created by Linear-GitHub Sync Desktop (Client) Created by Linear-GitHub Sync good first issue Good for newcomers High priority Created by Linear-GitHub Sync kind/bug Something isn't working web
Projects
None yet
Development

No branches or pull requests

2 participants