`#main-card-sortable > li > div`でもいいんですけど、もしもHTMLの構造を変えるときのことを考えると、
cardひとつひとつに共通したクラス名をつけておいて、`#main-card-sortable .card`みたいな感じにしておいたほうが、メンテナンスしやすいかもしれないですね。

あと` > li > div`よりも、何を指すかパッと見てわかるようになって、読みやすくなるかもしれないです。