Progress indicator not visible when scrollable is true and height not set

[Cory]

I've noticed that when I create a grid where scrollable is true (or default) and the height is not specified that the progress indicator is not visible. However, if I change the _progress function on the Grid to look like the following, the indicator is displayed and hidden as expected, regardless of whether/how scrollable or height is specified.

_progress: function(toggle) {
    var element = this.element;

    if (this.lockedContent) {
        element = this.wrapper;
    } else if (this.element.is("table")) {
        element = this.element.parent();
    } else if (this.content && this.content.length) {
        element = this.content.height() === 0
            ? this.content.parent()
            : this.content;
    }

    kendo.ui.progress(element, toggle);
}

From what I can tell, this produces the expected results in each scenario I've tested. Considering pull requests are not accepted for the Grid, I was wondering if someone can review this and, if acceptable, include in the next release.

[Dimo]

Hi Cory,

Thanks for the suggestion. We would like to refrain from measuring the data area height on each rebind. I recommend you to apply a min-height style to the .k-grid-content class in order to ensure that the loading indicator is always visible. A value of 50px will be sufficient.

Regards,
Dimo
Telerik

 

 

[Cory]

While your recommendation will allow the progress indicator to be displayed, it also leaves whitespace in the data area when there are no results. This becomes a different experience when compared to grids that have scrollable set to false, as the data area is collapsed so no whitespace is displayed (see attached).

[Dimo]

Hi Cory,

The fact that a Grid height is not set indicates that vertical scrolling is not important. If horizontal scrolling is not required either, then you can disable scrolling altogether, which will make the loading indicator display over the whole Grid.

In the vast majority of cases scrollable Grids are used with a height, which makes empty spaces possible if there are too few or no items. We do not regard this as an UI issue, as scrollability makes sense only together with predefined dimensions.

Regards,
Dimo
Telerik