cipherdevgroup/carelib

Layout name property is unused, but still being checked

Closed this issue · 2 comments

The recent changes to the layout API removed objects and stopped making use of the name property of the layouts. Instead, the layouts are named via their array key in an associative array.

Overall, this seems like it's working fine, but there are a couple of instances where the name property is still being checked. In these cases, the name is always empty and this is causing some weirdness when trying to change layouts from the admin or the customizer.

I think that last commit should resolve the issue, but we need to do some more testing around this before we tag 1.0 and our themes hit the public.

Closing this out for now. We can deal with bugs on another issue if they arise.