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

Template Parts: Inconsistent replace/edit sidebar views #60677

Open
bgardner opened this issue Apr 11, 2024 · 5 comments
Open

Template Parts: Inconsistent replace/edit sidebar views #60677

bgardner opened this issue Apr 11, 2024 · 5 comments
Labels
[Block] Template Part Affects the Template Parts Block [Type] Bug An existing feature does not function as intended

Comments

@bgardner
Copy link

bgardner commented Apr 11, 2024

Description

While testing #60156, I noticed that there was an inconsistent view/text in the right sidebar.

Step-by-step reproduction instructions

  1. Go to Appearance > Editor > Templates > Index in the site editor.
  2. Click the Header template part and view sidebar.
  3. Click Edit on the Header template part and view sidebar.

Screenshots, screen recording, code snippet

When you complete step #2 above, this is the sidebar:

image

Then when you complete step #3 above, this is the sidebar:

image

It seems as though the styling (spacing between icon and Header) and the copy should be consistent.

Below is a video walkthrough of the experience:

Screen.Recording.2024-04-11.at.10.23.11.AM.mov

Environment info

WordPress 6.5.2, Gutenberg 18.1.

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

@bgardner bgardner added the [Type] Bug An existing feature does not function as intended label Apr 11, 2024
@bgardner bgardner changed the title Template Parts: Inconsistent replace/edit views Template Parts: Inconsistent replace/edit sidebar views Apr 11, 2024
@Mamaduka Mamaduka added the [Block] Template Part Affects the Template Parts Block label Apr 11, 2024
@colorful-tones
Copy link
Member

I almost missed this key distinction on the first pass when reading this:

It seems as though the styling (spacing between icon and Header) and the copy should be consistent.

This is the bug 😄

Template Part block selected Template Part block in edit mode
321687933-579823c8-b659-4d75-b7fa-46a952b084a4 321688129-88bf66ca-bee1-4344-b0b3-e8efb53f1a38

@colorful-tones colorful-tones self-assigned this Apr 11, 2024
@bgardner
Copy link
Author

bgardner commented Apr 11, 2024

@colorful-tones It is a bug, yes. But... and more importantly, note below that section is not consistent: Design vs. Transform into: (with a paragraph of text below it)

^ this is the part I was trying to emphasize (lack of consistency)

@bgardner
Copy link
Author

bgardner commented Apr 11, 2024

Per:
image

image

@colorful-tones
Copy link
Member

Here is the current experience on Gutenberg Trunk c86e77f

template-part-flow-language.mp4

@colorful-tones colorful-tones removed their assignment Apr 11, 2024
@colorful-tones
Copy link
Member

This sent me down a rabbit hole 🕳️ 🐇 of trying to figure out how/why/what/where the overarching changes driving the changes referenced in this bug were stemming from, and I'm still not certain. The pull request Brian references: #60156 suggests it is merely:

Updates the "Replace" label to "Design". Suggested in #59883

and

The designs in #59689 suggest a "Design" panel for this purpose

If one tries to follow the path to the next referenced pull request #59883 they'll find it was closed and explored elsewhere (yet, another pull request):

Thanks for the discussion on this. I am trying a new idea in #60203

So, when you go to #60203, and you find it is merely referencing the previous "exploration" in the previously closed pull request: #59883 🤕

So, I stumbled across this "Proposal" Issue #59970, which seems to be partly tied to maybe what was trying to be addressed by #60156, #59883, and #60203. Hopefully, somebody more privy to what goes on around here can dig in faster with any of this information 💀 🤞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Template Part Affects the Template Parts Block [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

No branches or pull requests

3 participants