Skip to content

Conversation

eduardo-proteinms
Copy link
Collaborator

@eduardo-proteinms eduardo-proteinms commented Sep 25, 2024

Updated the DocumentConverter project to support post processing of html for issues that Brendan has been identifying. This should be safe to run multiple times.
 

node .\lib\skyline-cli.js --options '
{
        ""sourcePath"": ""C:/Users/Eduardo/repos/pwiz/pwiz_tools/Skyline/Documentation/Tutorials"",
        ""postProcessOnly"": true
}'

Currently this does the following:

  • Replace strong and em tags with b/i
  • Remove b/i tags when only used for comma, period, colon
  • Remove b/i tags that just contain whitespace, but preserve whitespace
  • Move trailing periods, colons or commas outside of b/i tags
  • Move leading and trailing whitespace to outside of b/i tags
  • Remove any empty b/i tags
  • Remove empty lines before p closing tags
  • Remove empty anchor tags(these come from word anchoring and are not needed)

@brendanx67
Copy link
Contributor

This is going to have to be redone after I merge PR 3171.

@eduardo-proteinms
Copy link
Collaborator Author

@brendanx67 I reapplied the post processing. this should be good to merge, but let me know your thoughts.

@brendanx67 brendanx67 merged commit bc60c96 into master Sep 28, 2024
7 of 9 checks passed
@brendanx67 brendanx67 deleted the Skyline/work/20240922_TutorialPostProccessing branch January 3, 2025 23:40
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