On 25.7.2014 22:27, Endi Sukma Dewata wrote:
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.
Pushed to:
ipa-4-0: ee61651bc9667462dec8e6dd2e64dbfeb249deed
ipa-4-1: 9aed114d822efb0eaa01d93624bc0ea6612c4169
master: 9aed114d822efb0eaa01d93624bc0ea6612c4169
--
Petr Vobornik
_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel
