Skip to content

DOC-14481-update-cb-intro - #899

Merged
iamfortune merged 3 commits into
masterfrom
DOC-14481-update-cb-mobile-page
Jul 22, 2026
Merged

DOC-14481-update-cb-intro#899
iamfortune merged 3 commits into
masterfrom
DOC-14481-update-cb-mobile-page

Conversation

@iamfortune

@iamfortune iamfortune commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Docs Issue: DOC-14481

PR to update the Mobile intro page

Preview URL:
https://preview.docs-test.couchbase.com/docs-site-DOC-14481-update-cb-mobile-page/home/mobile.html

You will need the Docs Team credentials on Confluence.

Comment thread home/modules/ROOT/pages/mobile.adoc Outdated
Comment thread home/modules/ROOT/pages/mobile.adoc Outdated
@iamfortune
iamfortune requested a review from idulo July 13, 2026 10:49
@iamfortune
iamfortune requested review from idulo and osfameron July 17, 2026 16:52

Couchbase Mobile supports three deployment patterns.

In a self-managed deployment, Couchbase Lite connects to a Sync Gateway that you deploy and operate, and Sync Gateway pushes data to a self-managed Couchbase Server.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this could be bulleted or numbered list


++++
<div class="card-row">
++++

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't you need this markup for the layout to work correctly?
I notice you still have the </div> later on


++++
<div class="card-row">
++++

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

although... this would have been a good occasion to consider moving to the more "modern" card layout we use for e.g. Capella landing pages.

Not urgent, but something to add to roadmap perhaps

@osfameron osfameron left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I see, there used to be 2 card layout rows, and you've removed the first one.
So my previous objection isn't valid.

I've checked the preview, and it looks fine (but do consider updating the visual layout of the page at some point in future!)

Happy to approve.

@iamfortune
iamfortune merged commit 8513424 into master Jul 22, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants