chore: Moves the stylesheets folder to the assets folder (#16880)

* chore: Moves the stylesheets folder to the assets folder

* Changes {src} to src

superset-frontend/.eslintignore

@@ -19,9 +19,9 @@
 coverage/**
 dist/*
 src/assets/images/*
+src/assets/stylesheets/*
 node_modules/*
 node_modules*/*
-stylesheets/*
 vendor/*
 docs/*
 src/dashboard/deprecated/*

superset-frontend/package.json

@@ -19,10 +19,10 @@
     "build-instrumented": "cross-env NODE_ENV=development BABEL_ENV=instrumented webpack --mode=development --color",
     "build": "cross-env NODE_OPTIONS=--max_old_space_size=8192 NODE_ENV=production BABEL_ENV=\"${BABEL_ENV:=production}\" webpack --mode=production --color",
     "lint": "eslint --ignore-path=.eslintignore --ext .js,.jsx,.ts,.tsx . && npm run type",
-    "prettier-check": "prettier --check '{src,stylesheets}/**/*.{css,less,sass,scss}'",
+    "prettier-check": "prettier --check 'src/**/*.{css,less,sass,scss}'",
     "lint-fix": "eslint --fix --ignore-path=.eslintignore --ext .js,.jsx,.ts,tsx . && npm run clean-css && npm run type",
-    "clean-css": "prettier --write '{src,stylesheets}/**/*.{css,less,sass,scss}'",
-    "format": "prettier --write './{src,spec,stylesheets,cypress-base}/**/*{.js,.jsx,.ts,.tsx,.css,.less,.scss,.sass}'",
+    "clean-css": "prettier --write 'src/**/*.{css,less,sass,scss}'",
+    "format": "prettier --write './{src,spec,cypress-base}/**/*{.js,.jsx,.ts,.tsx,.css,.less,.scss,.sass}'",
     "prettier": "npm run format",
     "check-translation": "prettier --check ../superset/translations/**/LC_MESSAGES/*.json",
     "clean-translation": "prettier --write ../superset/translations/**/LC_MESSAGES/*.json",

superset-frontend/src/CRUD/crud.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../stylesheets/less/variables.less';
+@import '../assets/stylesheets/less/variables.less';
 
 .CRUD {
   .text-right {

superset-frontend/src/SqlLab/App.jsx

@@ -39,7 +39,7 @@ import { BYTES_PER_CHAR, KB_STORAGE } from './constants';
 import setupApp from '../setup/setupApp';
 
 import './main.less';
-import '../../stylesheets/reactable-pagination.less';
+import '../assets/stylesheets/reactable-pagination.less';
 import '../components/FilterableTable/FilterableTableStyles.less';
 import { theme } from '../preamble';
 

superset-frontend/src/SqlLab/main.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../stylesheets/less/variables.less';
+@import '../assets/stylesheets/less/variables.less';
 
 body {
   min-height: ~'max(100vh, 500px)'; // Set a min height so the gutter is always visible when resizing

superset-frontend/src/assets/stylesheets/less/cosmo/cosmoTheme.js

@@ -16,6 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
+/* eslint-disable */
 (function () {
   $(window).scroll(function () {
     var top = $(document).scrollTop();

superset-frontend/src/assets/stylesheets/reactable-pagination.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../stylesheets/less/variables.less';
+@import './less/variables.less';
 
 .reactable-pagination td {
   padding: 15px 0 0 0 !important;

superset-frontend/src/assets/stylesheets/superset.less

@@ -579,6 +579,6 @@ hr {
   border-top: 1px solid @gray-light;
 }
 .ace_gutter-cell.ace_error {
-  background-image: url('../src/assets/images/icons/error_solid_small_red.svg') !important;
+  background-image: url('../images/icons/error_solid_small_red.svg') !important;
   background-position: -2px center !important;
 }

superset-frontend/src/components/FilterableTable/FilterableTableStyles.less

@@ -17,7 +17,7 @@
  * under the License.
  */
 
-@import '../../../stylesheets/less/variables.less';
+@import '../../assets/stylesheets/less/variables.less';
 
 .ReactVirtualized__Grid__innerScrollContainer {
   border: 1px solid @gray-light;

superset-frontend/src/components/TableLoader/index.tsx

@@ -21,7 +21,7 @@ import { t, SupersetClient, JsonObject } from '@superset-ui/core';
 import TableView, { EmptyWrapperType } from 'src/components/TableView';
 import withToasts from 'src/components/MessageToasts/withToasts';
 import Loading from 'src/components/Loading';
-import 'stylesheets/reactable-pagination.less';
+import '../../assets/stylesheets/reactable-pagination.less';
 
 export interface TableLoaderProps {
   dataEndpoint?: string;

superset-frontend/src/dashboard/stylesheets/builder-sidepane.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../../stylesheets/less/variables.less';
+@import '../../assets/stylesheets/less/variables.less';
 
 .dashboard-builder-sidepane {
   flex: 0 0 @builder-pane-width;

superset-frontend/src/dashboard/stylesheets/components/new-component.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../../../stylesheets/less/variables.less';
+@import '../../../assets/stylesheets/less/variables.less';
 
 .new-component {
   display: flex;

superset-frontend/src/dashboard/stylesheets/filter-scope-selector.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../../stylesheets/less/variables.less';
+@import '../../assets/stylesheets/less/variables.less';
 
 .filter-scope-container {
   display: flex;

superset-frontend/src/dashboard/stylesheets/index.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import './../../../stylesheets/less/variables.less';
+@import '../../assets/stylesheets/less/variables.less';
 
 @import './builder.less';
 @import './builder-sidepane.less';

superset-frontend/src/explore/App.jsx

@@ -27,7 +27,7 @@ import ToastContainer from 'src/components/MessageToasts/ToastContainer';
 import setupApp from 'src/setup/setupApp';
 import setupPlugins from 'src/setup/setupPlugins';
 import './main.less';
-import 'stylesheets/reactable-pagination.less';
+import '../assets/stylesheets/reactable-pagination.less';
 import { theme } from 'src/preamble';
 import ExploreViewContainer from './components/ExploreViewContainer';
 

superset-frontend/src/explore/main.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../stylesheets/less/variables.less';
+@import '../assets/stylesheets/less/variables.less';
 
 .scrollbar-container {
   position: relative;

superset-frontend/src/profile/main.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../stylesheets/less/variables.less';
+@import '../assets/stylesheets/less/variables.less';
 
 .tab-pane {
   min-height: 400px;

superset-frontend/src/theme.ts

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-import '../stylesheets/superset.less';
+import './assets/stylesheets/superset.less';
 
 // Importing Antd under its own stylesheet to prevent unintentional theming.
-import '../stylesheets/antd/index.less';
+import './assets/stylesheets/antd/index.less';

superset-frontend/src/utils/downloadAsImage.ts

@@ -24,7 +24,7 @@ import { addWarningToast } from 'src/components/MessageToasts/actions';
 
 /**
  * @remark
- * same as https://github.com/apache/superset/blob/c53bc4ddf9808a8bb6916bbe3cb31935d33a2420/superset-frontend/stylesheets/less/variables.less#L34
+ * same as https://github.com/apache/superset/blob/c53bc4ddf9808a8bb6916bbe3cb31935d33a2420/superset-frontend/src/assets/stylesheets/less/variables.less#L34
  */
 const GRAY_BACKGROUND_COLOR = '#F5F5F5';
 

superset-frontend/src/visualizations/FilterBox/FilterBox.less

@@ -16,7 +16,7 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-@import '../../../stylesheets/less/variables.less';
+@import '../../assets/stylesheets/less/variables.less';
 
 .dashboard .filter_box .slice_container > div:not(.alert) {
   padding-top: 0;

superset-frontend/webpack.config.js

@@ -129,7 +129,7 @@ const plugins = [
     patterns: [
       'package.json',
       { from: 'src/assets/images', to: 'images' },
-      { from: 'stylesheets', to: 'stylesheets' },
+      { from: 'src/assets/stylesheets', to: 'stylesheets' },
     ],
   }),