yari icon indicating copy to clipboard operation
yari copied to clipboard

Sidebars created by QuicklinksWithSubPages have duplicate entries

Open chrisdavidmills opened this issue 1 year ago • 5 comments

Summary

Sidebars created using the QuicklinksWithSubPages macro look really bad, and they contain duplicate links. See https://developer.mozilla.org/en-US/docs/Web/Privacy/Firefox_tracking_protection for an example.

We are about to update this page, so also see https://github.com/mdn/content/pull/30107#discussion_r1498492464 for a screenshot of what it used to look like.

URL

https://developer.mozilla.org/en-US/docs/Web/Privacy/Firefox_tracking_protection

Reproduction steps

  1. Go to https://developer.mozilla.org/en-US/docs/Web/Privacy/Firefox_tracking_protection

Expected behavior

A nice sidebar, without duplicate links

Actual behavior

A sidebar that looks bad, has duplicate links, and is inconsistent with other MDN sidebars in look and feel.

Device

Desktop

Browser

Firefox

Browser version

Pre-release (e.g. Beta, Nightly or Canary)

Operating system

Mac OS

Screenshot

No response

Anything else?

No response

Validations

chrisdavidmills avatar Feb 22 '24 17:02 chrisdavidmills

@chrisdavidmills Is this still an issue? The page mentioned looks fine, no duplicates in the sidebar.

argl avatar Mar 04 '24 19:03 argl

@chrisdavidmills Is this still an issue? The page mentioned looks fine, no duplicates in the sidebar.

@argl This is because we merged the content PR where the issue was first noticed, and it now uses a different mechanism of creating the sidebar. You can see what it used to look like (when borked) at this comment: https://github.com/mdn/content/pull/30107#discussion_r1498492464

chrisdavidmills avatar Mar 05 '24 10:03 chrisdavidmills

@chrisdavidmills Ok, understood, thank you!

argl avatar Mar 05 '24 13:03 argl

I think we should indeed create a PrivacySidebar macro for consistency, which can then also be used in the translated pages (see here and here).

caugner avatar Mar 15 '24 11:03 caugner

Noticed a potential side effect of this. On pages with duplicate sidebar entries like https://developer.mozilla.org/en-US/docs/Web/Privacy/Privacy_sandbox or https://developer.mozilla.org/en-US/docs/Web/EXSLT/math, when selecting one of those topics the whole section gets highlighted. image This does not happen when selecting one of the sub-entries in that section. image

mirunacurtean avatar May 21 '24 20:05 mirunacurtean