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

Site Editor: unnatural animation when switching to edit mode #62563

Open
t-hamano opened this issue Jun 14, 2024 · 5 comments
Open

Site Editor: unnatural animation when switching to edit mode #62563

t-hamano opened this issue Jun 14, 2024 · 5 comments
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Needs design efforts. [Type] Bug An existing feature does not function as intended

Comments

@t-hamano
Copy link
Contributor

Description

When you select an editable item from the Data View, the Canvas appears, squashing the Data View into the Canvas and making animations look unnatural.

Step-by-step reproduction instructions

In the data view, click an editable template or pattern.

Screenshots, screen recording, code snippet

Below is a video demonstrating the issue.

default.mp4

Here is a slowed-down version of the video to make the issue more visible.

normal.mp4

In WordPress 6.5, the details page exists and this problem does not occur.

b17b01cd33bd68c3637269de00dac123.mp4

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@t-hamano t-hamano added [Type] Bug An existing feature does not function as intended [Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") labels Jun 14, 2024
@stokesman
Copy link
Contributor

Good issue. I've noticed this too. I suppose what might be a natural animation would be if the canvas could start with the placement and scale of the preview. For cases (like table view) where there is no preview it might at least start with the placement of the data view.

@annezazu
Copy link
Contributor

@youknowriad could this be related to your PR? #61579

Please provide the environment you all are using btw @t-hamano @stokesman :)

@stokesman
Copy link
Contributor

stokesman commented Jun 18, 2024

I'm seeing it on trunk plugin installed on WP 6.5.2 / Chrome 124 / macOS.

could this be related to your PR? #61579

I'd taken a look at this and there is a difference that might be called worse but the animation was poor beforehand. Here’s what I see rolling back to before that PR. The difference is the data view vanishes instead of getting squashed. The canvas still starts from the same place.

transition-to-edit-from-grid-view-pre-61579.mp4

@youknowriad
Copy link
Contributor

youknowriad commented Jun 18, 2024

I don't think we have a good understand of what transition we want in this case. When the preview frame is not visible before the transition. I would love some design feedback/direction before moving into implementation... cc @jasmussen @jameskoster

@youknowriad youknowriad added the Needs Design Needs design efforts. label Jun 20, 2024
@jameskoster
Copy link
Contributor

Yes this needs some design / prototyping to get right.

My feeling is that the content frame (where the data view lives) is separate from the preview/editor frame, and each element lives on a different layer in terms of elevation.

layout

This structure should inform the animation effect. In data views, when the preview frame isn't visible, we can consider it conceptually present, but visually off-screen. Clicking 'edit' brings it into view, above the data view:

editor

I'm curious to hear how others think about this though, and there are other details to consider such as which 'plain' the Inspector lives on (given we want it to be accessible in both the editor and in data views).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Site Editor Related to the overarching Site Editor (formerly "full site editing") Needs Design Needs design efforts. [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

5 participants