[Freeipa-devel] [PATCH] 712 webui: detach facet nodes

[Endi Sukma Dewata]

On 7/23/2014 8:25 AM, Petr Vobornik wrote:
> Detach/attach facet nodes when switching facets instead of
> hiding/showing.
>
> Keeps dom-tree more simple.
>
>
> This patch is not really needed. I implemented it while testing
> something in IE. But it might have positive effect for poorly written
> parts of Web UI(if there are any :) ) or plugins. Basically it
> simplifies DOM tree to contain nodes only for the active facet.
> Therefore ugly expressions like $('button .foobar') are much more
> performant.

ACK. In the future the entire facet itself probably can even be loaded 
dynamically, so this is a step in that direction. The "facet" element 
itself probably can be merged with the "content" element since there's 
only one facet/content at any time.

-- 
Endi S. Dewata