Path variable for images not fully functional


Topic: Path variable for images not fully functional

admin5 pro asked 7 years ago

We are getting there XD

I refer back to my previous questions:

Can we get path variable in Sass
Can we get path variable in Sass (ASKING AGAIN)

In the latest version you have introduced the image-path variable:
$image-path: '../img/';

There is however 2 problems with this:

  1. It is not $image-path: '../img/' !default; Which means that I can't override it without changing the mdb.scss file. The !default needs to be added.
  2. In _masks.scss you link to images without using the newly introduced image variable background: url('../img/overlays/#{$filename}.png'); should be background: url('#{$image-path}/overlays/#{$filename}.png');

I have just tested the free version (because I'm on a open source project) installed via npm install mdbootstrap, so there could be more occurrences of this issue in the paid version.


a.mroczek commented 7 years ago

I've changes category because MDB jQuery team is responsible for scss files.

Marta Szymanska staff pro premium answered 7 years ago

Hi, we fixed this in MDB 4.5.1 version. Best, Marta

Gunnar Gabrielson commented 7 years ago

Just FYI it is not fixed as of yet in MDBReact / Pro.

Marta Szymanska staff pro premium answered 7 years ago

Hi, I see this but we can change this only in next release, we can't change sass files in MDB pack right now. Best, Marta

ghengeli pro commented 7 years ago

So glad to see this issue STILL EXISTS SIX MONTHS LATER...

Marta Szymanska staff pro premium commented 7 years ago

Hi, our sass code is still changing and we still improving it.

Gunnar Gabrielson commented 7 years ago

Yes I have this same issue. I am compiling the sass out of my node_modules and am able to change the font path variable, but as the original poster said, the $image-path does not have the default flag and the overlays also remain an issue. This issue was reported 8 months ago... any chance that this ever gets fixed?

Marta Szymanska staff pro premium answered 7 years ago

Hi, for MDB React package it will be fixed in the next release of MDB React in May. Best, Marta
Please insert min. 20 characters.

FREE CONSULTATION

Hire our experts to build a dedicated project. We'll analyze your business requirements, for free.

Status

Answered

Specification of the issue
  • User: Pro
  • Premium support: No
  • Technology: MDB jQuery
  • MDB Version: -
  • Device: -
  • Browser: -
  • OS: -
  • Provided sample code: No
  • Provided link: Yes