DataTable: Not displayed when utilizing Reflow

Description: DataTables with “reflow=true” are not shown when nested within an element that contains “style=display: none;”. When attempting to show the hidden element, the style attribute on the DataTable is set to: null;visibility:hidden;

@melloware, I believe this bug was introduced while attempting to resolve Issue: #7050, #7038 with Commit: 52071270f5f710b0add34fdd4748ac1174b66a4d

Reproducer primefaces-test.zip

Environment:

• PF Version: 11.0.0
• PF Theme: Saga
• JSF + version: Mojarra 2.2.20
• Affected browsers: ALL

To Reproduce Steps to reproduce the behavior:

1. Build / Run the attached project.
2. Click on ‘Show Section’
3. See bug

Expected behavior I’d expect both the DataTable with “reflow=true” and without “reflow” to be shown.

Actual behavior Only the table without “reflow=true” is shown.

Example XHTML

<p:commandButton onclick="$('#section').show();" type="button" value="Show Section"/>
<div id="section" style="display: none;">
    <h1>Reflow:</h1>
    <p:dataTable reflow="true" ...>
    ...
    </p:dataTable>

    <h1>No Reflow:</h1>
    <p:dataTable ...>
    ...              
    </p:dataTable>
</div>

primefaces-test.zip

[melloware]

I can’t believe this hasn’t come up before either but I think its because most people are not loading their components to the page with display:none…they are doing an update="section" on the DIV around the table and basically letting AJAX load it lazily when you click the Show button. You are preloading yours into the DOM only to then try and show it. so my guess its not a common use case because its not quite the “JSF way” of doing things.

[melloware]

Workaround attached: pf-8614.zip

Just do this onclick="PF('tblReflow').jq.css('visibility', 'visible');$('#section').show();"

However @tandraschko I think there is a bigger issue here I don’t quite get. The Datatable _render method is NEVER called so there is actually a bigger bug here than just this. Basially nothing in the _render method gets called so this means in the scenario above where it starts display:none none of the setup code for sticky, scrolling etc would ever get set up.

I have a breakpoint in _render and it never gets called because in renderDeferred the jq.is(":visible") is of course false.