Less live build #946

Unito
reinhard ha unito 11 commit da feature/dev-less a master 2019-05-29 09:03:28 +00:00
Membro

What

This PR introduces a live build of the less files in the browser.

It removes the fsfe.min.css from the repo.

Why

This should solve some issues regarding styles:

  • It isn't necessary any longer to call lessc for development
    • That was an obstacle for some people
  • Conflicts in the fsfe.min.css file are avoided

How

There is now a new file config.xsl in the project root. Depending on the css-mode variable it uses the live browser less build or includes the fsfe.min.css.

browser → less browser build
compiled → include fsfe.min.css

ToDo

There are still some things to do to make this work.

  • Less build on build/deployment system
  • Some testing
**What** This PR introduces a live build of the less files in the browser. It removes the `fsfe.min.css` from the repo. **Why** This should solve some issues regarding styles: * It isn't necessary any longer to call `lessc` for development * That was an obstacle for some people * Conflicts in the `fsfe.min.css` file are avoided **How** There is now a new file `config.xsl` in the project root. Depending on the `css-mode` variable it uses the live browser less build or includes the `fsfe.min.css`. `browser` → less browser build `compiled` → include `fsfe.min.css` **ToDo** There are still some things to do to make this work. * [ ] Less build on build/deployment system * [ ] Some testing
mweimann ha aggiunto l'etichetta
build
2019-05-25 17:29:04 +00:00
Autore
Membro

@reinhard maybe you could have a look at the idea. For the build system setup I also need your help (see ToDo).

cc @max.mehl

@reinhard maybe you could have a look at the idea. For the build system setup I also need your help (see ToDo). cc @max.mehl
Membro

How cool!

To build the .min.css files, I think the main (phase 1) Makefile would be the ideal place.

There might be better ways to distingiush between development and production environment, for example by adding a xslt parameter which is automatically set in the build script. I first need to understand which actually should be development environment, and which should be production environment. For example, will the automatic build of test.fsfe.org be production or development?

How cool! To build the .min.css files, I think the main (phase 1) Makefile would be the ideal place. There might be better ways to distingiush between development and production environment, for example by adding a xslt parameter which is automatically set in the build script. I first need to understand which actually should be development environment, and which should be production environment. For example, will the automatic build of test.fsfe.org be production or development?
Autore
Membro

@reinhard I took another round with Max to integrate the valentine mode as well. We updated the name of the control file to config. The variable is now css-mode. Max suggested to simply sed the css-mode in the make file. Could you prepare that.

@reinhard I took another round with Max to integrate the valentine mode as well. We updated the name of the control file to `config`. The variable is now `css-mode`. Max suggested to simply `sed` the `css-mode` in the make file. Could you prepare that.
Proprietario

Max suggested to simply sed the css-mode in the make file

Sorry, I didn't see Reinhard's suggestion before, a parameter would be even cooler, but to be honest I don't have a strong preference here.

For example, will the automatic build of test.fsfe.org be production or development?

IMHO production or compiled as it is called now. The server can take care of the CSS generation, so I see no reason for increasing the loading time.

> Max suggested to simply sed the css-mode in the make file Sorry, I didn't see Reinhard's suggestion before, a parameter would be even cooler, but to be honest I don't have a strong preference here. > For example, will the automatic build of test.fsfe.org be production or development? IMHO production or `compiled` as it is called now. The server can take care of the CSS generation, so I see no reason for increasing the loading time.
Membro

So the dynamic genereration of the CSS with JavaScript is intended only for the case when fsfe-local-build is used with fsfe-preview.sh, correct?

So the dynamic genereration of the CSS with JavaScript is intended only for the case when fsfe-local-build is used with fsfe-preview.sh, correct?
Autore
Membro

Yes, it's only for local development.

Yes, it's only for local development.
Membro

Great feature, thank you! Just a reminder that once this gets merged we should update this documentation.

Great feature, thank you! Just a reminder that once this gets merged we should update [this](http://wiki.fsfe.org/TechDocs/Mainpage/CSSwithLESS) documentation.
mweimann Titolo modificato da WIP: Less live build a Less live build 2019-05-27 18:27:25 +00:00
Autore
Membro

@reinhard if you're happy with it could you prepare the make file? lessc then has also to be available on the build system.

I'll update the wiki page right after the merge.

@reinhard if you're happy with it could you prepare the make file? `lessc` then has also to be available on the build system. I'll update the wiki page right after the merge.
Membro

Sorry, I was busy today, will comment tomorrow.

Sorry, I was busy today, will comment tomorrow.
reinhard chiusa questa pull request 2019-05-29 09:03:28 +00:00
Proprietario

Really nice, thank you @mweimann and @reinhard !

For all people who already use local build, it's important to run fsfe-preview --copy look/ build/ scripts/ to have everything prepared. From then on, all pages have to be rebuild at least once to render them correctly since fsfe.min.css is no longer available. But probably a full rebuild would make sense anyway.

Really nice, thank you @mweimann and @reinhard ! For all people who already use local build, it's important to run `fsfe-preview --copy look/ build/ scripts/` to have everything prepared. From then on, all pages have to be rebuild at least once to render them correctly since fsfe.min.css is no longer available. But probably a full rebuild would make sense anyway.
Autore
Membro

I wrote a ticket for updating the wiki and send a mail to the list: #967

I'll do that later.

I wrote a ticket for updating the wiki and send a mail to the list: #967 I'll do that later.
mweimann branch feature/dev-less eliminato 2019-05-29 10:46:53 +00:00
max.mehl aggiunta alle pietre miliari Hackathon1905 2019-06-03 14:28:38 +00:00
Effettua l'accesso per partecipare alla conversazione.
No description provided.