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.

Sign up for GitHub

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

Jump to bottom

ContentNegotiator class improvements #10980

Open

sabina-talipova opened this issue Oct 17, 2023 · 0 comments

Labels

affects/v5 type/enhancement

Contributor

sabina-talipova commented Oct 17, 2023 •

edited by GuySartorelli

Loading

Description

Check for correct XHTML doctype in xhtml().
ContentNegotiator::xhtml: Search for more xhtml replacement. See
Allow for all valid HTML4+ doctypes (e.g. Transitional) in html()
Make content replacement and doctype setting two separately configurable behaviours

See:

silverstripe-framework/src/Control/ContentNegotiator.php

Lines 28 to 30 in cbd358a

    
            * @todo Check for correct XHTML doctype in xhtml() 
        
            * @todo Allow for other HTML4 doctypes (e.g. Transitional) in html() 
        
            * @todo Make content replacement and doctype setting two separately configurable behaviours

The text was updated successfully, but these errors were encountered:

sabina-talipova added type/enhancement affects/v5 labels

sabina-talipova mentioned this issue

Remove all TODO comments in the code silverstripe/.github#116

Closed

GuySartorelli changed the title ~~TODO: ContentNegotiator class improvements~~ ContentNegotiator class improvements

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