-
Notifications
You must be signed in to change notification settings - Fork 164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rebuild main admin page #2445
Comments
…/caldera-forms into project/admin-refactor * 'project/admin-refactor' of https://github.com/CalderaWP/caldera-forms: hacky solution to avoid loading old assets in admin #2445 Hi roy commit for new admin app #2445 restart admin client # Conflicts: # package.json # vendor/autoload.php # vendor/calderawp/caldera-containers/.travis.yml # vendor/calderawp/caldera-containers/README.MD # vendor/calderawp/caldera-containers/Tests/ServiceServiceContainerTest.php # vendor/calderawp/caldera-containers/composer.json # vendor/calderawp/caldera-containers/src/Service/Container.php # vendor/calderawp/caldera-forms-query # vendor/composer/autoload_classmap.php # vendor/composer/autoload_real.php # vendor/composer/autoload_static.php # vendor/composer/installed.json
OK, I'm ready to hand this one to you. I copied in most of the plugin I was using to develop this, we are no longer using that repo. Just Caldera Forms.
Todos for @Shelob9 after 05d8c74
|
Pretty sure to add CSS compilation from
and also fix the saas loader. https://github.com/zgordon/how-to-gutenberg-plugin/blob/master/webpack.config.js |
For the tabs |
@Shelob9 This generates the build version of index.scss, the clients/admin/build/style.min.css still needs to be enqueued and I'm not sure if we want to enqueue css files from the clients folder or move theme and enqueue from the assets folder. BTW, the admin client is loading these styles via import within the index.js file, once we decide where we enqueue css from we can remove the import.
@New0 here is the new mockup I promised: These are notes corresponding to the numbers in the image:
Also:
|
Some initial work will get merged with #2444 but not used.
The text was updated successfully, but these errors were encountered: