Fix card layout for cards without cover

[zenoachtig]

I accidentally broke card layouts without covers, here's a fix

[changeset-bot]

⚠️ No Changeset found

Latest commit: 2590edd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[github-actions]

Summary of the deployments:

Version 1 (production)

Version	URL	Status
Latest commit	https://a2f4947f.gitbook-open.pages.dev	✅
PR	https://pr2831.gitbook-open.pages.dev	✅

Version 2 (experimental)

Version	URL	Status
Vercel	https://gitbook-v2-l73af2odu-gitbook.vercel.app	✅
Cloudflare		✅

Test content

Site	v1	v2
GitBook	https://a2f4947f.gitbook-open.pages.dev/docs.gitbook.com	https://gitbook-v2-l73af2odu-gitbook.vercel.app/static/url/docs.gitbook.com
E2E	https://a2f4947f.gitbook-open.pages.dev/gitbook.gitbook.io/test-gitbook-open	https://gitbook-v2-l73af2odu-gitbook.vercel.app/static/url/gitbook.gitbook.io/test-gitbook-open

[argos-ci]

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build	Status	Details	Updated (UTC)
default (Inspect)	👍 Changes approved	9 changed	Feb 13, 2025, 1:40 PM

[valentin0h]

Suggested change

	cover
	cover && coverIsLandscape

[zenoachtig]

No I want none of these classes when there is no cover – combining the statements would still render the portrait classes if there is no cover (that's the current bug)

If there's a better way to write this logic statement I'm open to it, I don't like this double ternary

[valentin0h]

ha I missed the null

[taranvohra]

Nested ternaries are a pain for readability but I guess it's fine here