Fix mobile version of ReviewList on the example demo

[alanpoulain]

Checking the loading prop for rendering the SimpleList was wrong, it should be !loading or loaded.

The ReviewMobileList component in the demo has also been fixed (it's coming from the SimpleList component).

[fzaninotto]

In SimpleList, at that stage, the loaded prop is always true, because of the previous test:

    if (loaded === false) {
        return (
            <SimpleListLoading
                classes={classes}
                className={className}
                hasLeftAvatarOrIcon={!!leftIcon || !!leftAvatar}
                hasRightAvatarOrIcon={!!rightIcon || !!rightAvatar}
                hasSecondaryText={!!secondaryText}
                hasTertiaryText={!!tertiaryText}
            />
        );
    }

    return (
        (loading || total > 0) && (
            <List className={className} {...sanitizeListRestProps(rest)}>
...

So a test for the loaded value is useless.

The question is: is the test for the loading value useful? There is a case where both loading and loaded can be true: when fetching the data for the list after it's already been rendered once (i.e. when coming back to a list page already viewed). The current test forces the display of the list in that case even though the total is null. I don't see the point.

So you can remove the test on the loading value altogether in SimpleList.

I agree with the change on ReviewListMobile.

[fzaninotto]

Thanks!