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

On-hover dialogues don't disappear on Linux #973

Closed
Blazing-Blast opened this issue Jan 6, 2024 · 4 comments
Closed

On-hover dialogues don't disappear on Linux #973

Blazing-Blast opened this issue Jan 6, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@Blazing-Blast
Copy link

Describe the bug

On Linux I have the issue that all of the on-hover dialogues don't disappear when I remove my mouse from them.

Steps to reproduce

  1. Open launcher.
  2. Hover mouse over a button (such as Home, Browse, Library, Log in/ -out).
  3. A little box will pop-up, with one or two words.
  4. Move mouse somewhere else.
  5. The box will stay.

Expected behavior

Instead of step 5, the box will go away.

System information

uname -a
Linux blazingblast 6.7.0-rc8-00130-g6d0dc8559c84 #52 SMP PREEMPT Fri Jan 5 20:37:00 CET 2024 x86_64 GNU/Linux
pacman -Q modrinth-app-bin
modrinth-app-bin 0.6.3-1

Additional context

image

@Blazing-Blast Blazing-Blast added the bug Something isn't working label Jan 6, 2024
@triphora
Copy link
Contributor

triphora commented Jan 6, 2024

I can reproduce this, but it seems to only occur when it's active while navigating to another page.

@tobeqz
Copy link

tobeqz commented Jan 18, 2024

This happens to me almost every time (but not always) in the AUR package modrinth-app and when building the project in release mode, but for some reason I can't reproduce it when I'm in pnpm tauri dev. Maybe something weird is going on with webkit2gtk?

@tobeqz
Copy link

tobeqz commented Jan 18, 2024

I compiled the release version with debug assertions enabled. I can reproduce this bug by opening the app and immediately clicking one of my profiles. It seems like an error is being thrown somewhere in popper.js internals which causes this to happen.
image
image

I tried to set a breakpoint on uncaught exceptions to figure out what exactly is being passed into Reflect.get (my guess is
undefined), but this bug is actual nightmare fuel and I cannot reproduce it while DevTools are open.

Curiously, the AppImage bundle generated by the same compilation command seems to work perfectly fine.

Reloading the page does work as a workaround, but unfortunately, AFAIK there is no way to reload the page in the release builds available in the AUR. I think it would be nice to give users the option to reload for situations like this and if it's wanted I can add a hotkey like Ctrl+R or F5.

@triphora
Copy link
Contributor

Duplicate of #829

@triphora triphora marked this as a duplicate of #829 Jan 27, 2024
@triphora triphora closed this as not planned Won't fix, can't repro, duplicate, stale Jan 27, 2024
Geometrically added a commit that referenced this issue Jul 4, 2024
* Migrate to Nuxt 3

* Update vercel config

* remove tsconfig comment

* Changelog experiment + working proj pages

* Fix package json

* Prevent vercel complaining

* fix deploy (hopefully)

* Tag generator

* Switch to yarn

* Vercel pls 🙏

* Fix tag generation bug

* Make (most) non-logged in pages work

* fix base build

* Linting + state

* Eradicate axios, make most user pages work

* Fix checkbox state being set incorrectly

* Make most things work

* Final stretch

* Finish (most) things

* Move to update model value

* Fix modal text getting blurred from transforms (#964)

* Adjust nav-link border radius when focused (#961)

* Transition between animation states on TextLogo (#955)

* Transition between animation states on TextLogo

* Remove unused refs

* Fixes from review

* Disable tabbing to pagination arrows when disabled (#972)

* Make position of the "no results" text on grid/gallery views consistent (fixes #963) (#965)

* Fix position of the "no results" text on grid view

* fix padding

* Remove extra margin on main page, fixes #957 (#959)

* Fix layout shift and placeholder line height (#973)

* Fix a lot of issues

* Fix more nuxt 3 issues

* fix not all versions showing up (temp)

* inline inter css file

* More nuxt 3 fixes

* [skip ci] broken- backup changes

* Change modpack warnings to blue instead of red (#991)

* Fix some hydration issues

* Update nuxt

* Fix some images not showing

* Add pagination to versions page + fix lag

* Make changelog page consistent with versions page

* sync before merge

* Delete old file

* Fix actions failing

* update branch

* Fixes navbar transition animation. (#1012)

* Fixes navbar transition animation.

* Fixes Y-axis animation. Fixes mobile menu. Removes highlightjs prop.

* Changes xss call to renderString.

* Fixes renderString call.

* Removes unnecessary styling.

* Reverts mobile nav change.

* Nuxt 3 Lazy Loading Search (#1022)

* Uses lazyFetch for results. onSearchChange refreshes. Adds loading circle.

* Removes console.log

* Preserves old page when paging.

* Diagnosing filtering bugs.

* Fix single facet filtering

* Implements useAuth in settings/account.

* tiny ssr fix

* Updating nuxt.config checklist.

* Implements useAuth in revenue, moneitzation, and dashboard index pages.

* Fixes setups.

* Eliminates results when path changes. Adds animated logo.

* Ensures loading animation renders on search page.

---------

Co-authored-by: Jai A <jaiagr+gpg@pm.me>

* Fix navigation issues

* Square button fix (#1023)

* Removes checklist from nuxt.config.

* Modifies Nuxt CI to build after linting.

* Fixes prettierignore file.

* bug fixes

* Update whitelist domains

* Page improvements, fix CLS

* Fix a lot of things

* Fix project type redirect

* Fix 404 errors

* Fix user settings + hydration error

* Final fixes

* fix(creator-section): border radius on icons not aligning with bg (#1027)

Co-authored-by: MagnusHJensen <magnus.holm.jensen@lego.dk>

* Improvements to the mobile navbar (#984)

* Transition between animation states on TextLogo

* Remove unused refs

* Fixes from review

* Improvements to the mobile nav menu

* fix avatar alt text

* Nevermind, got confused for a moment

* Tab bar, menu layout improvements

* Highlight search icon when menu is open

* Update layouts/default.vue

Co-authored-by: Magnus Jensen <magnushjensen.mail@gmail.com>

* Fix some issues

* Use caret instead

* Run prettier

* Add create a project

---------

Co-authored-by: Magnus Jensen <magnushjensen.mail@gmail.com>
Co-authored-by: Geometrically <18202329+Geometrically@users.noreply.github.com>
Co-authored-by: Jai A <jaiagr+gpg@pm.me>

* Fix mobile menu issues

* More issues

* Fix lint

---------

Co-authored-by: Kaeden Murphy <kmurphy@kaedenmurphy.dev>
Co-authored-by: triphora <emmaffle@modrinth.com>
Co-authored-by: Zach Baird <30800863+ZachBaird@users.noreply.github.com>
Co-authored-by: stairman06 <36215135+stairman06@users.noreply.github.com>
Co-authored-by: Zachary Baird <zdb1994@yahoo.com>
Co-authored-by: Magnus Jensen <magnushjensen.mail@gmail.com>
Co-authored-by: MagnusHJensen <magnus.holm.jensen@lego.dk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

3 participants