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

Multiple "Edit" links when viewing the site as an admin #94358

Open
niranjan-uma-shankar opened this issue Sep 10, 2024 · 10 comments
Open

Multiple "Edit" links when viewing the site as an admin #94358

niranjan-uma-shankar opened this issue Sep 10, 2024 · 10 comments
Labels
[Feature] Post/Page Editor The editor for editing posts and pages. [Pri] Normal Schedule for the next available opportuinity. Triaged To be used when issues have been triaged. [Type] Enhancement [Type] Fit and Finish Issues related to the fit, finish and overall polish of WordPress.com

Comments

@niranjan-uma-shankar
Copy link
Contributor

niranjan-uma-shankar commented Sep 10, 2024

Quick summary

This follows the comment made in p1725864908061989-slack-C029GN3KD.

(1) When viewing the site as an admin, there are five edit links. Should we remove some of these?

Screenshot 2024-09-09 at 12 17 21 PM

(2) The “Edit site” links on the page take to different destinations:

  • the one in the masterbar links to:
    https://{SITE_SLUG}.wordpress.com/wp-admin/site-editor.php?postType=wp_template&postId=pub/marl//header-footer-only&canvas=edit, which opens a page with an inserted template.
  • the one in the actionbar links to:
    https://{SITE_SLUG}.wordpress.com/wp-admin/site-editor.php?postType=wp_template&postId=pub/marl//header-footer-only
    which opens the template selector.

Both of them don’t seem to be apt destinations. It appears to me that we should be taking them to the root view - https://{SITE_SLUG}.wordpress.com/wp-admin/site-editor.php
which is what the Customizer links to.

(3) While we're at it, the site editor link in "My Home" can also be made to the point to the root view.

(4) Check #94358 (comment)

Steps to reproduce

  1. Create a new site and go to site at https://{SITE_SLUG}.wordpress.com
  2. Notice the multiple edit links.

Browser

No response

Other notes

No response

@jeherve
Copy link
Member

jeherve commented Sep 10, 2024

Moving this to the Calypso repository, since this repo is about the bug reporting tool, Bugomattic.

@jeherve jeherve transferred this issue from Automattic/bugomattic Sep 10, 2024
@github-actions github-actions bot added the [Pri] TBD Review and assign an appropriate [Pri] label as soon as possible. label Sep 10, 2024
@jeherve jeherve added [Type] Enhancement [Feature] Post/Page Editor The editor for editing posts and pages. and removed [Type] Bug labels Sep 10, 2024
@jeherve
Copy link
Member

jeherve commented Sep 10, 2024

The one element that seems most likely to be removable is the one we add via the custom top toolbar:

function EditorDocumentHead( { translate, siteId, type, typeObject, newPost } ) {

The 2 others are part of WordPress itself, and make sense to appear in the admin bar imo. They also link to 2 different editors.

@jeherve jeherve added [Pri] Normal Schedule for the next available opportuinity. Needs triage Ticket needs to be triaged and removed [Pri] TBD Review and assign an appropriate [Pri] label as soon as possible. labels Sep 10, 2024
@davemart-in
Copy link
Contributor

I agree with @jeherve.

@davemart-in
Copy link
Contributor

Here's what I shared in Slack:

I'd say, remove the one in the action bar and update the one in the masterbar to point to the root view.

@davemart-in davemart-in removed their assignment Sep 10, 2024
@davemart-in davemart-in moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Sep 10, 2024
@jeherve jeherve added Triaged To be used when issues have been triaged. and removed Needs triage Ticket needs to be triaged labels Sep 10, 2024
@richtabor
Copy link

I’d propose removing Edit Site from both the action bar and the master bar.

Generally, I think we’re pushing folks too often into the site editor directly without a clear intent (like customizing styles, or changing colors).

@niranjan-uma-shankar
Copy link
Contributor Author

niranjan-uma-shankar commented Sep 18, 2024

Is the appearance of the buttons theme specific? The screenshot in the issue description is of the Marl theme. On the "Fewer" theme, I see only the "Edit site" link:

Screenshot 2024-09-18 at 12 31 45 AM

@jeherve
Copy link
Member

jeherve commented Sep 18, 2024

I’d propose removing Edit Site from both the action bar and the master bar.

Wouldn't that introduce a discrepancy between Core and WordPress.com, something that we've been trying to reduce in the past few months? If the "Edit Site" button is confusing in the admin bar, maybe it should be removed from Core directly?

@mmtr mmtr self-assigned this Sep 25, 2024
@mmtr mmtr moved this from Triaged to In Progress in Automattic Prioritization: The One Board ™ Sep 25, 2024
@mmtr
Copy link
Member

mmtr commented Sep 25, 2024

I'd say, remove the one in the action bar and update the one in the masterbar to point to the root view.

I think "one" here refers to the "Edit site" link, right? This might have some unexpected implications for the action bar, because it would be empty for example when visiting the homepage:

Current Proposed
Screenshot 2024-09-25 at 13 25 50 Screenshot 2024-09-25 at 13 26 55

That action bar with just an ellipsis button looks quite wrong to me, so I'm moving this back to "Needs design" because I think we need to revisit how the action bar should look like under these circumstances.


Also, the initial report also indicated that the "Edit page" link (only visible when viewing a page) is duplicated across 3 sections:

  • Top toolbar
  • Launch banner (only on unlaunched sites)
  • Action bar

For consistency with the "Edit site" link, I think we want to remove it from both the action bar and the launch banner, and leave only the one in the top toolbar, which is what comes from Core.

@ramonjd
Copy link
Member

ramonjd commented Oct 20, 2024

This might have some unexpected implications for the action bar, because it would be empty for example when visiting the homepage:

That's the result I'm seeing in D162881-code too.

Why not remove the floating admin bar entirely, or redesign it to house whatever useful links it contains? E.g., stats/subs

What is the real value of the floating admin bar? Are there stats to speak to its benefit?

@Copons Copons added the [Type] Fit and Finish Issues related to the fit, finish and overall polish of WordPress.com label Nov 8, 2024
@matticbot matticbot moved this from Needs design to Triaged in Automattic Prioritization: The One Board ™ Nov 8, 2024
@richtabor
Copy link

richtabor commented Nov 8, 2024

That action bar with just an ellipsis button looks quite wrong to me, so I'm moving this back to "Needs design" because I think we need to revisit how the action bar should look like under these circumstances.

That's better than an Edit Site link that makes it too easy to break the site: #95363

What is the real value of the floating admin bar? Are there stats to speak to its benefit?

I too would like to validate the action bar in full. Are folks using it, successfully? Who can we reach out to do this? #96202

Why not remove the floating admin bar entirely, or redesign it to house whatever useful links it contains? E.g., stats/subs

I don't want to redesign auxiliary interfaces that WordPress already supports (admin bar for example).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Post/Page Editor The editor for editing posts and pages. [Pri] Normal Schedule for the next available opportuinity. Triaged To be used when issues have been triaged. [Type] Enhancement [Type] Fit and Finish Issues related to the fit, finish and overall polish of WordPress.com
Projects
Development

No branches or pull requests

7 participants