Tinymce 5 docs update pt. 2
Related Ticket:
Description of Changes:
- Integration options
- User interface options
- Content appearance options
- Content filtering options
Pre-checks:
- [x] Branch prefixed with
feature/orhotfix/ - [x] ~
_data/nav.ymlhas been updated (if applicable)~ - [x] ~Files has been included where required (if applicable)~
- [x] ~Files removed have been deleted, not just excluded from the build (if applicable)~
- [x] ~(New product features only) Release Note added~
Review:
- [x] Documentation Team Lead has reviewed
- [ ] Product Manager has reviewed
@noxuhax this has merge conflicts and LGTM is failing checks, can you please fix those before we review this?
@lnewson done. Sorry, forgot to mark it as a draft initially.
@lorenzo-pomili you are absolutely right, I somehow missed that links syntax I used is a legacy one. Went thru all configuration partials and updated it to xrefs.
some of those links have changed from absolute URL to in-page references, is that correct?
Yeah, since those partials are only used on one page and those links are all referencing elements on the same page, it does not make much sense to keep those links absolute IMO.
Yeah, since those partials are only used on one page and those links are all referencing elements on the same page, it does not make much sense to keep those links absolute IMO.
Regarding this, they are partials pages and can be used anywhere so the general rule is that they should be absolute xrefs otherwise they'll break when used in a page that doesn't include the needed partial. That said, given this is the TinyMCE 5 docs and aren't likely to change this might be fine 🤷