search

umco / umbraco-stacked-content

Stacked Content for Umbraco 7
https://our.umbraco.org/projects/backoffice-extensions/stacked-content
MIT License
17 stars 26 forks source link

Use ConfigureForDisplay to hideLabel #37

Closed mattbrailsford closed 6 years ago

mattbrailsford commented 6 years ago

I'm not sure if we are hiding the label if someone selects to hide it in the prevalue. Either way, we should be using ConfigureForDisplay like we do in nested content https://github.com/umco/umbraco-nested-content/blob/develop/src/Our.Umbraco.NestedContent/PropertyEditors/NestedContentPropertyEditor.cs#L105

leekelleher commented 6 years ago

Wondering if this is one of those that could belong in Inner Content? No pressure on that, just a passing thought.

mattbrailsford commented 6 years ago

I did ponder that myself. I think for the time being I'm happy to leave IC as just managing anything to do with content types as otherwise it could become a dumping ground for "useful" functions.