search

rubin-dp0 / Support

Submit Github Issues related to DP0
MIT License
1 stars 3 forks source link

[BUG] ADQL query box steals Command-` key combination in Mac OS #29

Closed wmwv closed 3 weeks ago

wmwv commented 2 years ago

Describe the bug A clear and concise description of what the bug is.

The ADQL Query box steals my "Alt-`" which I use to switch between windows of an application on Mac OS Monterey.

To Reproduce Steps to reproduce the behavior:

  1. Go to the Portal
  2. Click on 'Select Query Type'->'Edit ADQL (advanced)'
  3. Scroll down to 'ADQL Query'
  4. Click on the active text box. Now "Alt-`". You will get a pair of "``" in the textbox instead of switching to the next window of the application.

Expected behavior A clear and concise description of what you expected to happen.

I expected to switch to the next window of the application. If the cursor is not in the text box on that web page the switching works fine, but if the text box is open then you get pairs of "``".

Screenshots If applicable, add screenshots to help explain your problem.

Screen Shot 2022-06-28 at 14 28 15

URL If applicable, cut and paste the URL from your browser at which the issue occurred

https://data.lsst.cloud/portal/app/

Desktop (please complete the following information):

frossie commented 2 years ago

I am sorry - by Alt you mean the Command key (which is the normal MacOS window switching modifier?)

gpdf commented 2 years ago

From a macOS perspective, I assume you are talking about "Command-`"? "Alt" is more commonly mapped to macOS "Option".

I can reproduce this, if so.

wmwv commented 2 years ago

Sorry, "Command". (That's what I get for submitting bug reports while watching a workshop talk...)

MelissaGraham commented 1 year ago

@gpdf and @frossie, just checking in, is this a bug that can be fixed, and so the issue should stay open?

gpdf commented 1 year ago

We haven't had time to investigate this one yet, to determine whether the colorized-code widget that we are using is configurable to avoid this. So let's leave this open for a bit.

gpdf commented 1 year ago

This behavior is not currently present with Firefox. Will recheck shortly with the current version of Safari and then decide how to proceed further.

frossie commented 8 months ago

Sorry for the delay, this has been identified as a bug in a third party library, we have upstreamed a fix and we're waiting for it to shake down the release chain before we deploy it on data.lsst.cloud.

wmwv commented 8 months ago

Great! Thanks very much.

gpdf commented 3 weeks ago

My apologies for the late update. This was fixed in Firefly-2023.3.0 at the end of last year, and the fix was deployed in the last Portal version before the UI changes. This is also working correctly in the recent new-UI versions of the Portal.