cookiesEnabled function not present in the most recent version of theme-scripts/cart.js

[isaac-martin]

Sorry for crosspost - not sure if it belonged here or in the theme-scripts repo
Shopify starter and Slate have a function that looks like the below.

import {cookiesEnabled} from '@shopify/theme-cart';

// Apply a specific class to the html element for browser support of cookies.
if (cookiesEnabled()) {
  document.documentElement.className = document.documentElement.className.replace('supports-no-cookies', 'supports-cookies');
}

However the latest version of theme scripts / cart doesn't have that function cookiesEnabled exporting. This may be related to issue #5

[t-kelly]

Since the check for cookies it pretty straight forward and is not directly scoped to Shopify's Cart API, I decided not to include it in the v1 of theme-cart.

We definitely need to update this theme before upgrading to the latest version! Thanks.

[isaac-martin]

Hey @t-kelly i might open a PR for this one if you want?

[t-kelly]

Hey @isaac-martin -- please see Shopify/theme-scripts#65 first.

I'd be down to just put the contents of the the cookiesEnabled function directly in Starter Theme for now until we sort out where it can live in theme-scripts.

[isaac-martin]

Not a huge deal - on a bit of a code freeze until after Xmas so thought I'd put some work in here. I'll open an issue for something else I noticed and work on that instead if we give it the go ahead.

[Cam]

I'm using this for now:

// Apply a specific class to the html element for browser support of cookies.
if (window.navigator.cookieEnabled) {
  document.documentElement.className = document.documentElement.className.replace(
    'supports-no-cookies',
    'supports-cookies',
  );
}

Seems effective enough.