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

Referenced document won't open in group tab. #7265

Open
DavithkbY opened this issue Jul 29, 2024 · 8 comments · Fixed by #7394
Open

Referenced document won't open in group tab. #7265

DavithkbY opened this issue Jul 29, 2024 · 8 comments · Fixed by #7394
Assignees
Labels

Comments

@DavithkbY
Copy link

DavithkbY commented Jul 29, 2024

When trying to open a referenced document inside a group tab, the document doesn't open.

To Reproduce

Steps to reproduce the behavior:

  1. Clone this repo: https://github.com/DavithkbY/sanity-issue-demo-main
  2. Create a document
  3. Create a 'Document referencing document'
  4. Set a reference inside the 'navigations' tab and try to click on the document to open it in a pane
    Expected behavior

The referenced document should open in a new pane.
Currently this doesn't happen and is bugged.
It only opens when going to 'All Fields'.

Sanity Version

3.52.1

What operating system are you using?
Mac OS

Which versions of Node.js / npm are you running?

20.10.0

Video showing the bug:

Screen.Recording.2024-07-29.at.13.46.22.mov
@binoy14
Copy link
Contributor

binoy14 commented Aug 16, 2024

Apologies, we are working on fixing this

sjelfull added a commit that referenced this issue Aug 20, 2024
Removes blur handler on reference preview. This is a unnecessary footgun, and it is triggered because focus will change to the first element in the opened reference pane.

It isn’t clear why this only happens inside groups, though, so might dig more into that.

Fixes #7265

Signed-off-by: Fred Carlsen <[email protected]>
github-merge-queue bot pushed a commit that referenced this issue Aug 20, 2024
* fix(core): ignore all input/textarea elements when copying

fixes sdx-1591

Signed-off-by: Fred Carlsen <[email protected]>

* fix(core): fix blur issue for references inside groups

Removes blur handler on reference preview. This is a unnecessary footgun, and it is triggered because focus will change to the first element in the opened reference pane.

It isn’t clear why this only happens inside groups, though, so might dig more into that.

Fixes #7265

Signed-off-by: Fred Carlsen <[email protected]>

---------

Signed-off-by: Fred Carlsen <[email protected]>
@sjelfull
Copy link
Member

Thanks for reporting, a fix for this will be included in the next release.

@sjelfull sjelfull self-assigned this Aug 20, 2024
cngonzalez pushed a commit that referenced this issue Aug 20, 2024
* fix(core): ignore all input/textarea elements when copying

fixes sdx-1591

Signed-off-by: Fred Carlsen <[email protected]>

* fix(core): fix blur issue for references inside groups

Removes blur handler on reference preview. This is a unnecessary footgun, and it is triggered because focus will change to the first element in the opened reference pane.

It isn’t clear why this only happens inside groups, though, so might dig more into that.

Fixes #7265

Signed-off-by: Fred Carlsen <[email protected]>

---------

Signed-off-by: Fred Carlsen <[email protected]>
@DavithkbY
Copy link
Author

@sjelfull is the issue fixed? I still have it happening on my local project in sanity studio with the latest 3.55.0 version:

Screen.Recording.2024-08-26.at.13.24.56.mov

@DavithkbY
Copy link
Author

It's also still reproducible on the git repo. I've updated it to sanity 3.55.0. :

Screen.Recording.2024-08-26.at.13.29.51.mov

@ChBernat
Copy link

Bump, has this been fixed in 3.56? Can't see anything in the changelog.

jordanl17 pushed a commit that referenced this issue Aug 29, 2024
* fix(core): ignore all input/textarea elements when copying

fixes sdx-1591

Signed-off-by: Fred Carlsen <[email protected]>

* fix(core): fix blur issue for references inside groups

Removes blur handler on reference preview. This is a unnecessary footgun, and it is triggered because focus will change to the first element in the opened reference pane.

It isn’t clear why this only happens inside groups, though, so might dig more into that.

Fixes #7265

Signed-off-by: Fred Carlsen <[email protected]>

---------

Signed-off-by: Fred Carlsen <[email protected]>
@DavithkbY
Copy link
Author

@jordanl17 this issue still hasn't been fixed in Studio V 3.57.0

@DavithkbY
Copy link
Author

Bump, can this issue be reopened? It's still not fixed!

@sjelfull sjelfull reopened this Sep 16, 2024
@sjelfull
Copy link
Member

We are still looking into this. It is a bit tricky to find the root cause, since we are only able to reproduce it sometimes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants