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

Remove "Edit site" from the action bar #95363

Closed
richtabor opened this issue Oct 14, 2024 · 11 comments
Closed

Remove "Edit site" from the action bar #95363

richtabor opened this issue Oct 14, 2024 · 11 comments
Assignees
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request".

Comments

@richtabor
Copy link

An alternative to https://github.com/Automattic/dotcom-feedback/issues/302; let's just remove the "Site Editor" link from here.

The Site Editor should be treated more as an advanced toolset—not an omnipresent link in this "action bar".

Visual

Image

@ramonjd
Copy link
Member

ramonjd commented Oct 14, 2024

Running patch here: D162881-code

@richtabor
Copy link
Author

@ramonjd do you have an update on this? Thanks!

@ramonjd
Copy link
Member

ramonjd commented Nov 9, 2024

Thanks for the ping @richtabor

Latest is that the floating bar may be removed completely after #96022

See the comments from p58i-ioo-p2#comment-65033

cc @simison for confirmation

Either way if someone from @Automattic/dotcom-design has time to confirm whether D162881-code is still value we can pursue it in the meantime.

Copy link

github-actions bot commented Nov 9, 2024

Support References

This comment is automatically generated. Please do not edit it.

  • p58i-ioo-p2#comment-65033

@github-actions github-actions bot added the Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request". label Nov 9, 2024
@richtabor
Copy link
Author

Here's another example how rough this experience is, when you see an "Edit site" button, but you end up here:

CleanShot.2024-11-15.at.12.17.29.mp4

@ramonjd
Copy link
Member

ramonjd commented Nov 17, 2024

when you see an "Edit site" button, but you end up here

The first iteration of D162881-code (id 532692) fixed this. I think a non-controversial and easy win while we wait for a decision to remove this bar would be to fix this and just deploy. At least it would work.

@matt-west
Copy link
Contributor

Sounds like a sensible approach @ramonjd 👍

@ramonjd
Copy link
Member

ramonjd commented Nov 18, 2024

I think a non-controversial and easy win while we wait for a decision to remove this bar would be to fix this and just deploy. At least it would work.

I've reverted D162881-code to do this. If anyone has time to test, I'd appreciate it. Then we can move from there.

@jeryj
Copy link
Contributor

jeryj commented Jan 13, 2025

@ramonjd's approach is to fix the link to go to the specific template editor instead of the templates page. Is this a fine approach for now, or should we remove the entire "Edit Site" link? cc @richtabor

@ramonjd
Copy link
Member

ramonjd commented Jan 13, 2025

@jeryj I've got a PR ready to remove it for now.

169373-ghe-Automattic/wpcom

It's been approved so I'll commit it today or very soon.

We're also working through this list: #98042

There's a redesign proposal over in #94079 (comment)

@ramonjd
Copy link
Member

ramonjd commented Jan 16, 2025

Done via 169373-ghe-Automattic/wpcom

@ramonjd ramonjd closed this as completed Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Customer Report Issues or PRs that were reported via Happiness. Previously known as "Happiness Request".
Projects
None yet
Development

No branches or pull requests

5 participants