[vlc-devel] commit: qt4: remove b_is_node and use childCount() > 0 instead in playlist_model (Ilkka Ollakka )
jakob.leben at gmail.com
Mon Aug 17 22:31:19 CEST 2009
On Mon, Aug 17, 2009 at 10:18 PM, Ilkka Ollakka
<ilkka.ollakka+vlc at oamk.fi<ilkka.ollakka%2Bvlc at oamk.fi>
> Is there any real cases where it matters? eg why would there be
> significant nodes in playlist that has to show as nodes even if it's
> empty ?
This triggers a bigger issue: why is it possible to delete children of nodes
when it has no sense for a node to be empty?
But the point right now is that it is possible to empty a node and maybe you
want to fill it later again. Refilling it will not be possible as the node
will not be Qt::ItemIsDropEnabled anymore.
If you want to keep current implementation then we might just as well
disable deleting children of nodes. But I suggest we look forward to a day
when it will be possible to create arbitrary nodes with preffered names and
add/remove items to them at will.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel