remove extra semicolon

[LucienLeMagicien]

Webpack v5's CSS loader fails when seeing this double semicolon. The error looks something like:

 Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
TypeError: Cannot read property 'get' of undefined
    at /var/www/node_modules/webpack/lib/asset/AssetModulesPlugin.js:212:52
    [...]
error  in data:application/x-font-ttf;charset=utf-8;;base64,AAEAAAAOAIAAAwBgT1MvM[...]
Module build failed: UnhandledSchemeError: Reading from "data:application/x-font-ttf;charset=utf-8;;base64,AAEAAAAOAIAAAwBgT1MvM[...]" is not handled by plugins (Unhandled scheme).
Webpack supports "data:" and "file:" URIs by default.
You may need an additional plugin to handle "data:" URIs.

As @jlukic says in this comment, this should be the place to fix this. The contribution guide says to merge against next, but that branch is 32 commits behind so I'm not too sure. Please feel free to adapt.

If this ends up not being merged, or if it does not trigger the automated commits and release, I suppose the other way to fix this would be in the css parser used by webpack? Otherwise, workarounds can be found on the issues linked in the commit, namely:

• switching to Fomantic-UI: https://github.com/fomantic/Fomantic-UI/pull/2028
• using the @semantic-ui-react/css-patch package: https://github.com/Semantic-Org/Semantic-UI/issues/7073#issuecomment-1136392772
  • or more manually, using patch-package
• importing the css via a <style> tag instead of via webpack, since browsers seem to ignore the double semicolon: https://github.com/Semantic-Org/Semantic-UI-CSS/issues/75#issuecomment-1018326560
  • ie.:

    <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/semantic-ui/2.4.1/semantic.min.css" integrity="sha512-8bHTC73gkZ7rZ7vpqUQThUDhqcNFyYi2xgDgPDHc+GXVGHXq+xPjynxIopALmOPqzo9JZj0k6OqqewdGO3EsrQ==" crossorigin="anonymous" referrerpolicy="no-referrer" />

This would close #7073 once the updated css is published.

[jlukic]

I've gone ahead and merged this, will release to npm most likely this weekend.

[jlukic]

Think resolved in 2.5.0