Topic: Path variable for images not fully functional

admin5 pro asked 6 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.


free commented 6 years ago Edit Delete

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

Marta Wierzbicka staff answered 6 years ago


Hi, for MDB React package it will be fixed in the next release of MDB React in May. Best, Marta

Marta Wierzbicka staff answered 6 years ago


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

Gunnar Gabrielson free commented 6 years ago

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

Marta Wierzbicka staff answered 6 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 6 years ago

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

Marta Wierzbicka staff commented 6 years ago

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

Gunnar Gabrielson free commented 6 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?


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

  • ForumUser: Pro
  • Premium support: No
  • Technology: MDB jQuery
  • MDB Version: -
  • Device: -
  • Browser: -
  • OS: -
  • Provided sample code: Yes
  • Provided link: Yes
Tags