Skip to content

Comments

Improvements to chat message formatting#743

Open
Timwi wants to merge 1 commit intovck3000:masterfrom
Timwi:master
Open

Improvements to chat message formatting#743
Timwi wants to merge 1 commit intovck3000:masterfrom
Timwi:master

Conversation

@Timwi
Copy link

@Timwi Timwi commented Jun 10, 2025

  • Do all of the formatting in one pass, so you don’t get situations where a glossary term occurs inside of a URL and messes up the generated HTML.
  • Allow bare URLs as well as URLs enclosed in angle brackets (to allow more special characters, which the browser automatically UTF-8 escapes).
  • System is now more flexibly able to accommodate future things that may want to be HTMLified/linkified/adorned/etc.
  • Since this is now all handled in one place, call that place chatformat.js instead of glossary.js

* Do all of the formatting in one pass, so you don’t get situations where a glossary term occurs inside of a URL and messes up the generated HTML.
* Allow bare URLs as well as URLs enclosed in angle brackets (to allow more special characters, which the browser automatically UTF-8 escapes).
* System is now more flexibly able to accommodate future things that may want to be HTMLified/linkified/adorned/etc.
* Since this is now all handled in one place, call that place chatformat.js instead of glossary.js
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.

1 participant