ICEfaces
  1. ICEfaces
  2. ICE-8633

ice:outputResource displays incorrectly when used in an ice:tree

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Won't Fix
          • Affects Version/s: 3.1, EE-3.0.0.GA_P01, 3.2.BETA1
          • Fix Version/s: 3.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            Firefox, IE
          • Assignee Priority:
            P2

            Description


            When an ice:outputResource is used as a node for an ice:tree, the resource is displayed on a new line and is not inline with the tree icons. Even setting style="display: inline;" on the outputResource doesn't help.

            The main cause of this is that the values that are added to the style attribute are not being applied to the outer div that is rendered by the component.

              Activity

              Ascending order - Click to sort in descending order
              Migration created issue -
              Hide
              Permalink
              Arran Mccullough added a comment - - edited

              Attached test case that shows issue.

              Run welcomeICEfaces.jsf to see issue.

              Show
              Arran Mccullough added a comment - - edited Attached test case that shows issue. Run welcomeICEfaces.jsf to see issue.
              Migration made changes -
              Field Original Value New Value
              Reporter Migration [ remote ] Arran Mccullough [ arran.mccullough ]
              Migration made changes -
              Attachment tree-outputResource-issue.jpg [ 14961 ]
              Ken Fyten made changes -
              Assignee yip.ng [ yip.ng ]
              Fix Version/s 3.3 [ 10370 ]
              Assignee Priority P2 [ 10011 ]
              Hide
              Permalink
              yip.ng added a comment -

              What "attached test case?" Can't find it.

              Show
              yip.ng added a comment - What "attached test case?" Can't find it.
              Arran Mccullough made changes -
              Attachment Case11645Example.zip [ 15713 ]
              Attachment Case11645ExampleWAR.zip [ 15714 ]
              Hide
              Permalink
              yip.ng added a comment - - edited
              1. Same old issue of <div> (block display) versus <span> (inline display): root container is rendered as <div>.
              2. Same old confusion of where the style and styleClass should get rendered: both are not rendered at the root. (This confusion is one of the reasons I don't like the style and styleClass attributes. We had to deal with similar problems many times before.)

              Possible solutions:

              1. Render <span> instead of <div>? (High risk of breaking existing apps.)
              2. Render style and styleClass at the root? (Risk of breaking existing apps.)
              3. User CSS workaround as already mentioned in commented-out code in test app: 
                <style>
    .iceOutRsr{
        display: inline;
    }
</style>
              4. The above will affect all output resource instances. Better technique is use style class "scoping" as I mentioned before: screenshot-01.png. See my wiki article for more details.
              Show
              yip.ng added a comment - - edited Same old issue of <div> (block display) versus <span> (inline display): root container is rendered as <div>. Same old confusion of where the style and styleClass should get rendered: both are not rendered at the root. (This confusion is one of the reasons I don't like the style and styleClass attributes. We had to deal with similar problems many times before.) Possible solutions: Render <span> instead of <div>? (High risk of breaking existing apps.) Render style and styleClass at the root? (Risk of breaking existing apps.) User CSS workaround as already mentioned in commented-out code in test app: <style> .iceOutRsr{ display: inline; } </style> The above will affect all output resource instances. Better technique is use style class "scoping" as I mentioned before: screenshot-01.png . See my wiki article for more details.
              yip.ng made changes -
              Attachment screenshot-01.png [ 15715 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Due to the high risk of breaking backwards compatibility by adopting any of the possible fixes, marking Won't Fix.

              Recommendation is to use the documented application Work-around.

              Show
              Ken Fyten added a comment - Due to the high risk of breaking backwards compatibility by adopting any of the possible fixes, marking Won't Fix. Recommendation is to use the documented application Work-around.
              Ken Fyten made changes -
              Status Open [ 1 ] Closed [ 6 ]
              Resolution Won't Fix [ 2 ]

                People

                • Assignee:
                  yip.ng
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: