Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

missing CSS class for InPost integration #647

Open
kkolcon opened this issue Aug 6, 2024 · 7 comments
Open

missing CSS class for InPost integration #647

kkolcon opened this issue Aug 6, 2024 · 7 comments

Comments

@kkolcon
Copy link

kkolcon commented Aug 6, 2024

Hello,

I've noticed a problem with InPost free module - when module is installed and configured, map with Paczkomaty (Parcel Lockers) doesn't open in the shipping step.

The problem is caused by missing CSS class "carrier-extra-content" in file hummingbird/templates/checkout/_partials/steps/shipping.tpl, in line 61. This CSS class is used as a reference in the module code.
Here's how this part should look like (starting from line 60):

<div class="carrier__extra-content-wrapper {if $delivery_option == $carrier_id} carrier__extra-content-wrapper--active {/if} js-carrier-extra-content" {if $delivery_option !== $carrier_id}style="max-height:0px"{/if}>
   <div class="carrier__extra-content carrier-extra-content">
      {$carrier.extraContent nofilter}
   </div>
</div>
@SharakPL
Copy link
Contributor

One more example of BEM breaking things 😜

@kpodemski
Copy link
Contributor

Hello @kkolcon

It's this Inpost module that has to adapt to Hummingbird.

ping @Oksydan

@Oksydan
Copy link
Contributor

Oksydan commented Aug 20, 2024

Hi @kkolcon,

I'm not sure if we should be concerned about this right now 🤔.
You can send this information to InPost support and we can look into it, but I'd like to wait for the official stable version of the Hummingbird theme.

@kkolcon
Copy link
Author

kkolcon commented Aug 23, 2024

Hi,

I've already informed InPost and sent them my solution as well.
I just thought someone who use Hummingbird may come upon this problem, so I've written about it to reduce their debugging time 😀

@SharakPL
Copy link
Contributor

SharakPL commented Sep 2, 2024

It's this Inpost module that has to adapt to Hummingbird.

I don't get why we have to use BEM. I'm also pretty sure it won't be the only module with problems like that. Many of them refer to classes used typically on PS themes and they won't work on hummingbrid because standard-css-class was changed to standard--css__class. BEM is useless here!

Anyway, the fix is quite simple (add missing non-BEM class) so it should be fixed here.

@Hlavtox
Copy link
Contributor

Hlavtox commented Sep 2, 2024

@SharakPL BEM has nothing to do with some module relying on a class.

@SharakPL
Copy link
Contributor

SharakPL commented Sep 2, 2024

Sure, but completely unnecessary switch to BEM is the reason those modules don't work with Hummingbird.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants