Skip to content

chore: Replace markup_txt with rfc2html#6632

Merged
rjsparks merged 14 commits into
ietf-tools:mainfrom
larseggert:chore-remove-markup-txt-light
Jan 5, 2024
Merged

chore: Replace markup_txt with rfc2html#6632
rjsparks merged 14 commits into
ietf-tools:mainfrom
larseggert:chore-remove-markup-txt-light

Conversation

@larseggert

Copy link
Copy Markdown
Collaborator

This is a subset of #6629 that leaves the reStructured text stuff alone.

@rjsparks

Copy link
Copy Markdown
Member

This is a pretty major change, not just a library substitution for similar effect.

It essentially pulls a differently stylized htmlized view onto the main document page, replacing the plaintext view, getting use to even more confusion about where people should look.

We need to push for less, and that's a bigger conversation than what we should have in a PR.

@larseggert

Copy link
Copy Markdown
Collaborator Author

So I would be all in favor of completely removing the preview from this template, and force people to click the button for the rendering they want.

@larseggert

Copy link
Copy Markdown
Collaborator Author

Should I remove the preview?

@rjsparks

rjsparks commented Dec 8, 2023

Copy link
Copy Markdown
Member

We need to push for less, and that's a bigger conversation than what we should have in a PR.

I think we should have this discussion with the community (and really, this should be fodder for a more comprehensive UX-expert driven rework of the document display pages). Perhaps we should bring this up on tuesday's tools call.

@rjsparks

rjsparks commented Dec 8, 2023

Copy link
Copy Markdown
Member

To inform conversation Tuesday:
image
image

@rjsparks rjsparks left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Based on the 12Dec tools call discussion, we should try to push this forward, but it's failing several tests.

@rjsparks

Copy link
Copy Markdown
Member

It's now down to template validation failures - rfc2html may need tweaking?

@larseggert

Copy link
Copy Markdown
Collaborator Author

I have some validation ignores queued up for next week.

@codecov

codecov Bot commented Dec 18, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (02e0d96) 88.80% compared to head (584d5f1) 88.81%.
Report is 26 commits behind head on main.

Files Patch % Lines
ietf/doc/views_draft.py 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6632      +/-   ##
==========================================
+ Coverage   88.80%   88.81%   +0.01%     
==========================================
  Files         285      284       -1     
  Lines       40374    40361      -13     
==========================================
- Hits        35853    35846       -7     
+ Misses       4521     4515       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread docker/docker-compose.extend.yml Outdated
@rjsparks

rjsparks commented Jan 4, 2024

Copy link
Copy Markdown
Member

drafts for ads is still failing validation

@larseggert

Copy link
Copy Markdown
Collaborator Author

Hm. It passed after rebasing.

@rjsparks
rjsparks merged commit e24fb60 into ietf-tools:main Jan 5, 2024
@larseggert
larseggert deleted the chore-remove-markup-txt-light branch January 5, 2024 20:42
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jan 9, 2024
@jennifer-richards

Copy link
Copy Markdown
Member

@larseggert - the cross-reference URLs generated by this were broken. At /doc/draft-ietf-stir-certificates-ocsp/#page-11 (for example), the link for [RFC5280] is https://dt-main.dev.ietf.org/doc/draft-ietf-stir-certificates-ocsp/rfc5280 rather than https://dt-main.dev.ietf.org/doc/rfc5280

This will be reverted by #6909. Because we squash-merged this, you should be able to rebase this branch onto main after the revert PR is merged.

@jennifer-richards

Copy link
Copy Markdown
Member

I rebased the merged branch here: https://github.com/jennifer-richards/datatracker/tree/chore-remove-markup-txt-light

(You probably should double-check that I did that correctly as there were a few conflicts. Note that it does not include commits that did not leave any changes, so the commit count doesn't match this PR's 14 commits.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants