@vindl how do you feel about this change ?

cc @jeherve

Also cc @dereksmart

After some more thought, instead of disabling the toggle I think it would be even better if we removed the card completely.

instead of disabling the toggle I think it would be even better if we removed the card completely.

I would agree.

Self-note: Maybe it would be best to check if the available module is there instead of checking for atomic site.

@vindl @jeherve I think removing the settings card as a whole would demand more work in other pieces of the code than the PR that's already done. Don't know if I can do it for today's release though.

@MichaelArestad what do you think about this ?

I was just having this conversation in parallel with other folks. I would remove the Masterbar card altogether for now unless @apeatling has any objections.

@vindl @jeherve I think removing the settings card as a whole would demand more work in other pieces of the code than the PR that's already done. Don't know if I can do it for today's release though.

We should be able to keep all the changes contained in this file by bailing early in render:

if ( isAtomicSite ) {
    return null;
}

or

{ ! isAtomicSite && <SettingsCard ... }

Thanks for the feedback folks. I've addressed the requested changes. Will check in an Atomic site when the PR is available in Jetpack Beta Plugin. Merging afterwards if everything's ok!

Thanks @vindl . I've just checked in an Atomic Site. Merging now!