Skip to content

Conversation

lamATnginx
Copy link
Contributor

Proposed changes

Remove card-layout

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@lamATnginx lamATnginx self-assigned this Aug 19, 2025
@lamATnginx lamATnginx requested a review from a team as a code owner August 19, 2025 20:30
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nginx-plus Issues related to NGINX Plus (and NGINX Open Source in docs.nginx.com) labels Aug 19, 2025
Copy link
Contributor

@mjang mjang left a comment

Choose a reason for hiding this comment

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

IIRC, Jack noted in a previous PR that the broken build in PRs against the current mainframe branch are -- not an issue.

@lamATnginx
Copy link
Contributor Author

IIRC, Jack noted in a previous PR that the broken build in PRs against the current mainframe branch are -- not an issue.

We removed card-layout in the mainframe branch of the theme so this change is updating the landing page for it. We didn't update it on main branch theme cuz it can lead to some merge conflicts that overly complicate mainframe branch.

@lamATnginx lamATnginx merged commit 840b436 into mainframe Aug 19, 2025
6 of 7 checks passed
@lamATnginx lamATnginx deleted the plus-landing-cleanup branch August 19, 2025 22:38
nginx-jack pushed a commit that referenced this pull request Aug 20, 2025
feat: Remove card-layout from N+ landing page
nginx-jack pushed a commit that referenced this pull request Aug 20, 2025
feat: Remove card-layout from N+ landing page
nginx-jack pushed a commit that referenced this pull request Aug 22, 2025
feat: Remove card-layout from N+ landing page
nginx-jack pushed a commit that referenced this pull request Aug 25, 2025
feat: Remove card-layout from N+ landing page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation product/nginx-plus Issues related to NGINX Plus (and NGINX Open Source in docs.nginx.com)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants