Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.42k stars 1.99k forks source link

Hosting Management Panel: Improve Help Center anchors #94949

Open taipeicoder opened 3 weeks ago

taipeicoder commented 3 weeks ago

Quick summary

Currently, the cards of the Hosting Features tab matches the unlocked tabs once the site transfers to Atomic.

Screenshot 2024-09-26 at 10 05 31 PM Screenshot 2024-09-26 at 10 05 21 PM

Every card contains a "Learn more" link. The issue is that the Monitoring and Logs tab share the same support article, and thus has some odd anchoring when opening in the Help Center.

Monitoring Current Proposal
Screenshot 2024-09-26 at 10 06 05 PM Screenshot 2024-09-26 at 10 07 01 PM
Logs Current Proposal
Screenshot 2024-09-26 at 10 06 34 PM Screenshot 2024-09-26 at 10 06 55 PM

Steps to reproduce

  1. Go to wordpress.com/sites.
  2. Select a Simple site.
  3. Select the Hosting Features tab.
  4. Click on the Learn more links for the Monitoring, and Logs cards.

What you expected to happen

See screenshots in the Proposal column.

What actually happened

See screenshots in the Current column.

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

If the above answer is "Yes...", outline the workaround.

No response

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

fushar commented 3 weeks ago

This is because Monitoring and Logs were previously part of the same Calypso page /site-monitoring/:site, but in different tabs (Metrics , PHP Logs, and Webserver Logs IIRC).

I believe the correct solution here is to split the docs into two separate pages to better reflect the current state. Currently, the page still refers to the old structure:

image image
jartes commented 3 weeks ago

📌 REPRODUCTION RESULTS

📌 ACTIONS

📌 Message to Author @taipeicoder thanks for reporting this one. Good catch!

@fushar If the only solution here is to split the docs into two different pages, please, let me know so we can repot it to support repo.

taipeicoder commented 3 weeks ago

@ianstewart also recommended splitting the pages, so let's go with that. Thanks!

escapemanuele commented 3 weeks ago

Ok @taipeicoder! Splitting the pages and changing the links in this page will for sure solve this.

Of course the problem will still be there, as scrolling to the appropriate anchor should work anyway, but at least we won't have the problem in this page.