MSD: Add sidebar top padding when no logo

[p-jackson]

Fixes DOTMSD-1206

Proposed Changes

• Add padding-block-start: 24px to .dashboard-responsive-sidebar__sidebar when it has no __logo child, via :not(:has(...)).

Why are these changes being made?

CIAB (my.woo.ai) boots the dashboard w/ Logo: null. With dashboard/omnibar on, the dashboard's own <Header> is also skipped, so the first sidebar menu item rendered flush under the external omnibar — no breathing room. Dotcom is unaffected (always has a Logo).

CIAB may or may not eventually have a logo here, but while MSD supports it we should have some sort of way to handle having no logo.

Testing Instructions

• Enable dashboard/omnibar in config/dashboard-development.json.
• Load CIAB w/ omnibar on — confirm ~24px padding above "Stores" item.
• Load dotcom — confirm logo block still renders, no extra top padding stacked.
• Toggle omnibar off on CIAB — confirm no regression under own header.

Pre-merge Checklist

• Has the general commit checklist been followed? (PCYsg-hS-p2)
• Have you written new tests for your changes?
• Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
• Have you checked for TypeScript, React or other console errors?
• Have you tested accessibility for your changes? Ensure the feature remains usable with various user agents (e.g., browsers), interfaces (e.g., keyboard navigation), and assistive technologies (e.g., screen readers) (PCYsg-S3g-p2).
• Have you used memoizing on expensive computations?
• Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
  • For UI changes, have we tested the change in various languages (for example, ES, PT, FR, or DE)?
• For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

[github-actions]

Calypso Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-175061

Jetpack Cloud Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-175061&env=jetpack

Automattic for Agencies Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-175061&env=a8c-for-agencies

Dashboard Live (dotcom) (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-175061&env=dashboard

Dashboard Live (CIAB) (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-175061&env=dashboard-ciab

[matticbot]

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

• help-center
• notifications
• wpcom-block-editor

To test WordPress.com changes, run install-plugin.sh $pluginSlug ciab-msd-sidebar-top-logo on your sandbox.

[p-jackson]

I took this padding number from the existing flex gap value used between the horizontal rule and the nav items.

But it does mean that it's not even: the top and left/right paddings are different.

Not blocking I don't think, but do you have an opinion about how much padding to use between the omnibar and the first nav items when there's no logo @lucasmendes-design?

[StevenDufresne]

Code is fine. Looks like you ran into the intermittent build problem on calypso.live. Rebasing should fix.

[fushar]

I see. I think it's fine to merge this to avoid CIAB folks from seeing broken layout 🙈