feat: ability to conditionally remove ion-reorder from ion-item

[aperezbaena]

Type: feat

Ionic Version: 2.x

Platform: all

ion-item includes ion-reorder element in the DOM regardless of reorder="true" is set in the list, or even if it's not included in any ion-list or ion-item-group.

Having an option to prevent this item to be present in the DOM would lead to a cleaner and more performing output.

[manucorporat]

Have you noticed any performance problem caused by having ion-reorder in each item?

ion-reorder should be display: none when reorder is disabled, so it should not affect the layout or performance.

[aperezbaena]

As you say the whole ion-reorder element is hidden with display: none so it has no impact either in the layout or rendering. However all that unused markup must be parsed by the browser and kept in memory.

Couldn't measure up yet, and only a few items so far but it could have an impact with many items.

Besides for the sake of a cleaner DOM, what's the point yo have elements you know you're not going to use (ever)?

[manucorporat]

@aperezbaena-csc will be fixed in next release!

[aperezbaena]

@manucorporat awesome!!