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

Properly tag monospaced text in man pages. (Improves HTML rendering) #3509

Merged

Conversation

sebastiancarlos
Copy link
Contributor

@sebastiancarlos sebastiancarlos commented Jun 23, 2024

This would be immediately useful to improve the rendering of the man pages in third-party websites such as manned.org and the arch man pages:

The regular console output, or websites that render the manpage in monospaced fonts, will not be affected by this change.

This change could also help with eventually rendering the manpages in the documentation (GothenburgBitFactory/tw.org#794) website and mixing monospaced and variable-width fonts.

To test the HTML output:

git clone git@github.com:jacksonp/manner.git # used by ManKier
./manner/manner.php task.1.in >| task.1.html && $BROWSER task.1.html

Examples

Before

Screenshot 2024-06-23 at 12 19 30 PM ### After Screenshot 2024-06-23 at 12 19 48 PM

Copy link
Collaborator

@djmitche djmitche left a comment

Choose a reason for hiding this comment

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

Fantastic, thanks! Does task-sync.5.in need similar attention?

@djmitche
Copy link
Collaborator

Oh, I just saw

Note: I ignored task-sync.5 as it is deprecated.

That's not the case -- sync is not deprecated, just the old taskserver aka taskd.

This would be immediately useful to improve the rendering of the man
pages in third-party websites such as manned.org and the arch man pagesk
- https://mankier.com/1/task
- https://man.archlinux.org/man/task.1

The regular console output, or websites which render the manpage in
monospaced fonts, will not be affected by this change.

This change could also help with eventually rendering the manpages in
the documentation website, and having a mix of monospaced and variable
width fonts.

To test the HTML output:
```bash
git clone git@github.com:jacksonp/manner.git # used by ManKier
./manner/manner.php task.1.in >| task.1.html && $BROWSER task.1.html
```
@sebastiancarlos
Copy link
Contributor Author

That's not the case -- sync is not deprecated, just the old taskserver aka taskd.

Ah, I missed that. I did the same to task-sync.5.in now.

Copy link
Collaborator

@djmitche djmitche left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@djmitche djmitche merged commit 910860a into GothenburgBitFactory:develop Jun 24, 2024
14 checks passed
@sebastiancarlos sebastiancarlos deleted the man-monospace branch July 1, 2024 07:23
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