search

localgovdrupal / localgov_microsites

A repository for the LocalGov Drupal Microsites project.
GNU General Public License v2.0
3 stars 2 forks source link

Make summary field optional #95

Closed finnlewis closed 2 years ago

finnlewis commented 2 years ago

@stephen-cox

On https://localgov-micro-1.lndo.site/node/1/edit

The summary field is required.

Can we make it optional?

Image

Also @msayoung @richard-s-rowley @markconroy, assumig we keep the summary field, do we want to display the summary field on the node view or not?

stephen-cox commented 2 years ago

@finnlewis The summary also displays on teasers. I know we're not using them yet, but this needs to be considered. If we remove the summary what are we going to display on listing pages as such like? I don't think it's same to assume that Pages will never end up in a view

finnlewis commented 2 years ago

So you think we should keep it required and display it on the page? I guess that is what the help text says!

stephen-cox commented 2 years ago

I think it's fine as it is. It's a design decision to whether it's displayed on the page, but something will be needed for teaser views. If there's no summary to show, what should be displayed instead?

msayoung commented 2 years ago

Yes - it's clearer how it should be on the wireframes

Image

I might rework that now (though i'll put that on another ticket)

finnlewis commented 2 years ago

For now we keep it required,