GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding
https://grapesjs.com
Other
22.8k stars 4.12k forks source link

BUG: Duplicated CSS while multi page creation #5207

Closed gxanshu closed 1 year ago

gxanshu commented 1 year ago

GrapesJS version

What browser are you using?

Chrome latest

Reproducible demo link

https://codesandbox.io/s/withered-bush-wv29dx?file=/index.js

Describe the bug

How to reproduce the bug?

  1. ... create multi pages in grapesjs.init()
  2. ... check the styles of each page

What is the expected behavior? every page contains there own style not include the CSS of other pages ...

What is the current behavior? currently the CSS of all pages will merged and apply on all pages. for more details visit the reproduction link and check the code of all pages ...

If is necessary to execute some code in order to reproduce the bug, paste it here below:

// simple code for demonstration 
import grapesjs from "grapesjs";
import "grapesjs-blocks-basic";

const paperCode = [
  {
    css: "p{color:red}",
    html: "<p>first page</p>"
  },
  {
    css: "p{color:blue}",
    html: "<p>second page</p>"
  },
  {
    css: "p{color:green}",
    html: "<p>third page</p>"
  }
];

const editor = grapesjs.init({
  container: "#gjs",
  fromElement: 1,
  storageManager: { type: 0 },
  plugins: ["gjs-blocks-basic"],
  pageManager: {
    pages: paperCode.map((page, index) => {
      return {
        name: `page ${index + 1}`,
        id: `${index + 1}`,
        styles: page.css,
        component: page.html
      };
    })
  }
});

editor.Pages.select("1"); // in order to see other pages, change the value from 1 to 3. eg, "1", "2", "3"

Code of Conduct

artf commented 1 year ago

This is intentional and I'm not planning to change this behaviour as you can easily create specific CSS selectors if you need different styles per page. The big benefit of this approach is to allow reusing selectors across pages.

hasan-aa commented 2 months ago

I think I'm experiencing an issue with this approach. I'm saving the pages to server like this:

const component = page.getMainComponent();
const css = editor.getCss({ component, avoidProtected: true }) 
saveToBackendApi(component,css)

and loading them like this in a loop:

const page = pageManager.add({
    name: pageName,
    component: pageHtml || `<div> ${pageName} </div>`,
    styles:css,
});

The css rules targeting ids are working fine, but the rules with a class are duplicating even when loaded in separate pages. For instance the # of css rules for .gjs-row class keeps increasing after after each save + load.