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

Mouseup makes dialogs close #4639

Open
1 of 4 tasks
napter opened this issue Sep 8, 2022 · 3 comments
Open
1 of 4 tasks

Mouseup makes dialogs close #4639

napter opened this issue Sep 8, 2022 · 3 comments
Labels
approved Issue has been approved by the bot or manually for further processing frontend Related to the frontend improvement Improving functionality, behaviour, UX, ...

Comments

@napter
Copy link

napter commented Sep 8, 2022

The problem

This is mostly an inconvenience. If you highlight text in a dialog and the mouse moves off the dialog before the mouseup the dialog closes. This is common since textboxes are often near the edges of dialogs.

When it happens it is very confusing and despite knowing about the issue it still usually takes me a few tries before I realize what happened and then carefully select the text so that the final mouseup remains within the dialogs borders.

The dialog should only close from a mousedown outside of its borders, not a mouseup.

Did the issue persist even in safe mode?

Yes, it did persist

If you could not test in safe mode, please state why

No response

Version of OctoPrint

1.8.1

Operating system running OctoPrint

Windows

Printer model & used firmware incl. version

No response

Browser and version of browser, operating system running browser

No response

Checklist of files to include below

  • Systeminfo Bundle (always include!)
  • Contents of the JavaScript browser console (always include in cases of issues with the user interface)
  • Screenshots and/or videos showing the problem (always include in case of issues with the user interface)
  • GCODE file with which to reproduce (always include in case of issues with GCODE analysis or printing behaviour)

Additional information & file uploads

No response
octoprint-systeminfo-20220908205956.zip

@github-actions github-actions bot added the triage This issue needs triage label Sep 8, 2022
@jneilliii
Copy link
Contributor

I can confirm I've seen this before with some of my plugins that use bootstrap modal. I wonder if it's a potential issue with the library itself or if there's a setting that can be enabled on the modal call that would prevent this behavior.

@jneilliii jneilliii added improvement Improving functionality, behaviour, UX, ... frontend Related to the frontend approved Issue has been approved by the bot or manually for further processing labels Sep 8, 2022
@omkarshinge
Copy link

  • Regarding this issue I did a bit of research and found out that back then there are many bugs related to this issue in bootstrap source code .
  • By the way I'm new to the OctoPrint repository and found that version of bootstrap files used in the project are of v2.3.2 .
  • Here is the link to one such bug filed in bootstrap framework's source code : Link to the bug filed in v3 bootstrap source .
  • In above issue they have clearly stated that there won't be any fix provided by them for older versions (In the above issue it is mentioned v3.x) .
  • There's workaround mentioned too by an user to change the modal dismissing part of code to listen to mousedown event instead of click .
  • In newer version of Bootstrap (v5x)I guess the issue is fixed : Link to bug where issue is fixed .
  • So I think there are two options here either to upgrade bootstrap or to perform workaround and check if anything breaks due to the changes done .

@cp2004
Copy link
Member

cp2004 commented Oct 16, 2022

So I think there are two options here either to upgrade bootstrap or to perform workaround and check if anything breaks due to the changes done .

The first option (to upgrade bootstrap) is not an option for us - lots of discussion on this issue tracker & discord & probably the forums already as to why. We could make a change to fix the issue in the bundled bootstrap files to fix the issue or work around it.

@cp2004 cp2004 removed the triage This issue needs triage label Jan 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Issue has been approved by the bot or manually for further processing frontend Related to the frontend improvement Improving functionality, behaviour, UX, ...
Projects
Status: Todo
Development

No branches or pull requests

4 participants