Skip to content

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

Merged
Girgias merged 1 commit intophp:masterfrom
TimWolla:white-space-pre
Aug 16, 2023
Merged

Apply white-space: pre onto the entire code snippet#801
Girgias merged 1 commit intophp:masterfrom
TimWolla:white-space-pre

Conversation

@TimWolla
Copy link
Copy Markdown
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