Skip to content
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

Add a test for =pod Html #24

Closed
wants to merge 2 commits into from
Closed

Add a test for =pod Html #24

wants to merge 2 commits into from

Conversation

coke
Copy link

@coke coke commented Feb 2, 2017

Issue #23

@coke
Copy link
Author

coke commented Feb 2, 2017

Here's a very simplistic test, just making sure that no <p>'s were added to the code. (I expected that nothing would change, but this is a start)

@coke
Copy link
Author

coke commented Feb 3, 2017

Branch now has a test and a change to not update html/xhtml sections at all. All tests (including the new one) pass

@coke
Copy link
Author

coke commented Feb 3, 2017

I'm still not convinced this is 100% right, though.

Pass it through unchanged.
@tisonkun
Copy link
Member

tisonkun commented Nov 8, 2017

If all text inside =pod Html is raw HTML, it's right. Because Pod machinery wrap para into Pod::Block::Para.

However, users might using pod6 code inside Html block, which cause type object insert inside Html, like C<code> generate Pod::FormattingCode<140289445450176> . Thus I use node2rawtext instead, and now C<code> -> code. Anyway, do not using pod6 at all is recommended.

Remain the unescape in order to meet case when someone wants to write C<> or B<> literally for some reason. Although it results &lt; should be written as &amp;lt;, both of them are rare usage, possibility ahead convenience.

@tisonkun tisonkun closed this Nov 8, 2017
@JJ JJ deleted the coke/html-test branch October 26, 2020 11:26
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.

None yet

2 participants