Skip to content

Apply white-space: pre onto the entire code snippet #801

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

Merged
merged 1 commit into from
Aug 16, 2023
Merged

Apply white-space: pre onto the entire code snippet #801

merged 1 commit into from
Aug 16, 2023

Conversation

TimWolla
Copy link
Member

This is needed to correctly handle whitespace outside of a <?php … ?> portion.

see b7bccc4
see php/phd@01d6beb
see php/doc-en#2648

This is needed to correctly handle whitespace outside of a `<?php … ?>` portion.

see b7bccc4
see php/phd@01d6beb
see php/doc-en#2648
@Girgias Girgias merged commit 7d40388 into php:master Aug 16, 2023
@TimWolla TimWolla deleted the white-space-pre branch August 18, 2023 11:45
TimWolla added a commit to TimWolla/php-doc-en that referenced this pull request Aug 18, 2023
As per the discussion in php#2648, the part that might actually be
unexpected is that leading indentation also causes problems.  The addition of
the empty line distracts from that. Now that the CSS for PHP.net is fixed, this
can be reverted.

This reverts commit 1d73d6e.

see php/web-php#801
TimWolla added a commit to php/doc-en that referenced this pull request Aug 21, 2023
As per the discussion in #2648, the part that might actually be
unexpected is that leading indentation also causes problems.  The addition of
the empty line distracts from that. Now that the CSS for PHP.net is fixed, this
can be reverted.

This reverts commit 1d73d6e.

see php/web-php#801
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.

2 participants