Some people are confused by the options we give when we add a fullcalendar row style.
I made a patch to disabled some checkboxes when needed and added some clarification.

First I thought to make them #hidden but an empty fieldset looks kinda weird...
Maybe we could make them hidden but change the message on top a bit.

Dunno needs review, feel free to help :)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aspilicious’s picture

Status: Active » Needs review
aspilicious’s picture

This patch uses the hidden approach, I think it's better.

tim.plunkett’s picture

I love this idea, but when I create a new view and try to save the Row Style before adding fields, I get:
An illegal choice has been detected. Please contact the site administrator.
An illegal choice has been detected. Please contact the site administrator.
An illegal choice has been detected. Please contact the site administrator.

So, I combined the two patches, and it worked great. :) Can you double check this?

tim.plunkett’s picture

That was a rather heavy handed approach.
The checkboxes need to be hidden, the selects need to be disabled.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community

I noticed the ajax errors but I didn't care at that moment ;).
This works great :).

I love these small usability improvements. :)

tim.plunkett’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.