search

shipmondo / prestashop_shipping_module

Shipmondo Shipping Module for Prestashop
2 stars 4 forks source link

[Do not merge] Warehouse theme #44

Closed MathiasReker closed 5 years ago

MathiasReker commented 5 years ago

Ping @Tezting

Tezting commented 5 years ago

@MathiasReker Vi tænker det er mere korrekt at vi har enkelte CSS filer som overskriver "default shipmondo.css" afhænger af tema/plugin. Det gør at evt. opdateringer af pluginnet ikke vil slette ændringer du har lavet. Ved ikke om du blot har rettet i shipmondo.css for at teste?

MathiasReker commented 5 years ago

Jeg har blot rettet i shipmondo.css, men jeg kunne lige så godt overskrive css'en :-)

Jeg har faktisk lavet en tekst-editor i et andet PrestaShop modul, som kan overskrive kode. Hvis jeg får tid, vil jeg godt implementere den. Så vil der være et felt i BO, hvor man kan indsætte sin overskrivning. Denne overskrivning bliver så gemt i fx custom.css, som vil blive læst efter shipmondo.css.

Tezting commented 5 years ago

@MathiasReker Fx. css/1.7.6/warehouse.css css/1.7.6/shipmondo.css

Hvis så 1.8.0 vil kræve ændringer i vores/deres css for at se ordentlig ud vil vi oprette: css/1.8.0/warehouse.css css/1.8.0/shipmondo.css

Synes ikke vi skal have en editor indbygget :-)

MathiasReker commented 5 years ago

Det er også en mulighed.

MathiasReker commented 5 years ago

I praksis kan man ikke gøre noget, som er bullet-proof. Det må være sådan, at enten lever med default css'en, ellers må man rette den til :-)

Tezting commented 5 years ago

Vi ønsker en model så hvis en kunde benytter warehouse. Så detecter vi det og indlæser en custom css som overskriver vores base css.

MathiasReker commented 5 years ago

Ja, det var også den løsning som jeg havde i tankerne i første omgang.

MathiasReker commented 5 years ago

Koden kunne se ud i retning af dette: https://github.com/shipmondo/prestashop_shipping_module/pull/45

Så undgår vi case statements og det er nemt at tilføje nye temaer