ICEfaces
  1. ICEfaces
  2. ICE-5825

Several remaining xhtml compliance issues with compat components

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha3
          • Fix Version/s: 2.0.0
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 2 JSF 2
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration
          • Workaround Exists:
            Yes
          • Workaround Description:
            Hide
            Add the f:view tag with the following attribute to ensure that the content type is text/html:

            <f:view contentType="text/html">
            Show
            Add the f:view tag with the following attribute to ensure that the content type is text/html: <f:view contentType="text/html">

            Description

            Up to Alpha 3 release, the ICEfaces response writer (DOMResponseWriter) was returned hardcoded values for:

            content type: text/html
            encoding: UTF-8

            While this covers a wide range of use cases, it's not adaptable to change by application developers. Changes to the DOMResponseWriter in ICE-5664 - to provide better support for determining the proper content type and encoding - makes for a more compatible ICEfaces experience.

            However, now that the values are no longer hard coded, browsers can potentially handle the markup differently based on what they accept and prefer. For example Chrome and Safari currently prefer "application/xhtml+xml" over "text/html":

                Accept: application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5

            whereas Firefox prefers "text/html"

                Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8

            This is all fine (and better explained here - http://www.w3.org/TR/xhtml-media-types/) except that if the content type "application/xhtml-xml" is returned, then the default namespace must also be included or the page will not render correctly:

            <html xmlns="http://www.w3.org/1999/xhtml">

            So we need to ensure that we document this requirement and that our example applications are properly adjusted to ensure the default namespace is included in our page markup.

              Activity

              Ascending order - Click to sort in descending order
              Deryk Sinotte created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Salesforce Case []
              Fix Version/s 2.0-Beta [ 10231 ]
              Affects [Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration]
              Assignee Deryk Sinotte [ deryk.sinotte ]
              Ken Fyten made changes -
              Salesforce Case []
              Affects [Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration] [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration]
              Deryk Sinotte made changes -
              Summary Default xhtml namespace required Improve xhtml compliance
              Salesforce Case []
              Deryk Sinotte made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Deryk Sinotte made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Deryk Sinotte made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Deryk Sinotte made changes -
              Attachment ICE-5891-revert.patch [ 12443 ]
              Deryk Sinotte made changes -
              Salesforce Case []
              Component/s Bridge [ 10011 ]
              Component/s Components [ 10012 ]
              Assignee Deryk Sinotte [ deryk.sinotte ] Ken Fyten [ ken.fyten ]
              Ken Fyten made changes -
              Fix Version/s 2.0-Beta2 [ 10242 ]
              Fix Version/s 2.0-Beta1 [ 10231 ]
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 2.0.0 [ 10230 ]
              Fix Version/s 2.0-Beta2 [ 10242 ]
              Ken Fyten made changes -
              Summary Improve xhtml compliance Several remaining xhtml compliance issues with compat components
              Salesforce Case []
              Ken Fyten made changes -
              Salesforce Case []
              Component/s  Documentation [ 10018 ]
              Component/s Framework [ 10013 ]
              Component/s Sample Apps [ 10010 ]
              Component/s Bridge [ 10011 ]
              Deryk Sinotte made changes -
              Workaround Description Add the f:view tag with the following attribute to ensure that the content type is text/html:

              <f:view contentType="text/html">
              Workaround Exists [Yes]
              Salesforce Case []
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P2
              Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
              yip.ng made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              yip.ng made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P2

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Deryk Sinotte
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: