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

missing </p> tags #38

Closed
ghost opened this issue Jul 8, 2023 · 1 comment · Fixed by #39
Closed

missing </p> tags #38

ghost opened this issue Jul 8, 2023 · 1 comment · Fixed by #39

Comments

@ghost
Copy link

ghost commented Jul 8, 2023

the anchors are there, but the HTML is malformed:

<p class="level0">curl attempts to re-use connections when doing multiple file
transfers, so that getting many files from the same server do not use multiple
connects / handshakes. This improves speed. Connection re-use can only be done
for URLs specified for a single command line invocation and cannot be performed
between separate curl runs. <a name="OUTPUT"></a>
<h2 class="nroffsh">Output</h2>

notice carefully that the </p> is missing. this page:

https://curl.se/docs/manpage.html

docs:

@ghost
Copy link
Author

ghost commented Jul 8, 2023

error is here:

roffit/roffit

Lines 102 to 104 in 26d34eb

push @out, "\n<p class=\"level$indentlevel\">" if(!$pre);
push @out, @p;
}

shockingly, looks like error has existed since first commit 16 years ago:

e881000

@bagder bagder changed the title invalid HTML created missing </p> tags Jul 10, 2023
bagder added a commit that referenced this issue Jul 10, 2023
Also inhibit the entire <p> if there is no content

Update the test accordingly

Reported-by: Steven Penny
Fixes #38
bagder added a commit that referenced this issue Jul 10, 2023
Also inhibit the entire <p> if there is no content

Update the test accordingly

Reported-by: Steven Penny
Fixes #38
jhauga added a commit to jhauga/roffit that referenced this issue Jul 13, 2023
Made edits similar to 
[roffit bagder#39](bagder#39) that would 
reopen [roffit bagder#38](bagder#38).
jhauga added a commit to jhauga/roffit that referenced this issue Jul 13, 2023
* roffit: delete excessive test folder

* roffit: edit makefile comment on scripts

Edited Make file to use scripts, adding short descriptive comments 
for each of the scripts before they are called.

* roffit: add examples in test page

Included examples in test page that the supporting scripts used
in the Make file fix.

* roffit: update broken links script

Debugged script and added short descriptive comments.

* roffit: update fix special character script

Debugged script, adding short descriptive comments, and removing
prompt asking if there are options with special characters.

* roffit: change <STYLE> and additional </p>

Made edits similar to 
[roffit bagder#39](bagder#39) that would 
reopen [roffit bagder#38](bagder#38).

* roffit: update testpage.output with script fixes

Copied output from testpage.dump file to output to include changes from testpage.1
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 a pull request may close this issue.

0 participants