Skip to content

Move sass to devDependencies #3737

Closed
Closed
@GaelGirodon

Description

@GaelGirodon

This is about Bulma. It is an optimization/enhancement request.

Overview of the problem

This is about the Bulma CSS framework, more precisely about the published NPM package.

I'm using Bulma version 1.0.0.

This issue doesn't seem to be a duplicate.

Description

TL;DR Installing sass shouldn't be mandatory for developers using only CSS files.

  • Bulma v1 is a full rewrite of the framework using Dart Sass.
  • sass has been added as a dependency of the NPM package (to promote its usage with Bulma v1?), increasing dramatically the package size.
  • Compiled CSS is provided in the NPM package along Sass files, so Sass doesn't seem to be required in order to use Bulma. Even some customization can be done with CSS only (using CSS variables), Sass seems to be required only for advanced usage/customization.
  • sass dependency could be moved to devDependencies in order to reduce the package installation size, developers can still install sass when they need to use Bulma Sass files.

Steps to Reproduce

Install [email protected] NPM package (to use only CSS files).

Expected behavior

sass dependency installation shouldn't be forced when we only want to use Bulma CSS files.

Actual behavior

sass is a dependency of [email protected] and is installed automatically by NPM, even if uneeded.


P.S. Thanks for your work on the v1, it's great to see that this project is still alive 🎉

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions