Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Render feature comparisons with standard function #1612

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
theodorejb wants to merge 1 commit into php:master
base: master
Choose a base branch
Loading
from theodorejb:feature-comparisons-function

Conversation

@theodorejb
Copy link
Contributor

@theodorejb theodorejb commented Nov 16, 2025
edited
Loading

This refactor greatly simplifies maintaining consistent styling and markup for the feature comparisons.

No content has been changed, except for fixing unnecessary blank lines at the start & end of the "Command line linter supports multiple files" code blocks on the PHP 8.3 release page.

Copy link
Contributor

🚀 Preview for commit cf80cca can be found at https://web-php-pr-1612.preview.thephp.foundation

Copy link
Contributor

@theodorejb theodorejb force-pushed the feature-comparisons-function branch from d667638 to f5f8013 Compare November 16, 2025 15:10
This refactor greatly simplifies maintaining consistent styling and markup for the feature comparisons.
No content has been changed, except for fixing unnecessary blank lines at the start & end of the "Command line linter supports multiple files" code blocks on the PHP 8.3 release page.
@theodorejb theodorejb force-pushed the feature-comparisons-function branch from f5f8013 to 0aa67ea Compare November 19, 2025 21:38
Copy link
Member

derickr commented Dec 9, 2025

Is this still relevant, as the PHP 8.5 (and I suppose later) release announcements don't use this any more, and the existing ones are supposed to be static?

sy-records reacted with thumbs up emoji

Copy link
Contributor Author

It simplifies maintaining the pages in a consistent way. If we want to update the whole site to a new design, this will make it much easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@saundefined saundefined Awaiting requested review from saundefined saundefined is a code owner

@sy-records sy-records Awaiting requested review from sy-records sy-records is a code owner

@sgolemon sgolemon Awaiting requested review from sgolemon sgolemon is a code owner

@derickr derickr Awaiting requested review from derickr derickr is a code owner

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /