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

Remove regular price span to show price label #1139

Closed
wants to merge 1 commit into from
Closed

Remove regular price span to show price label #1139

wants to merge 1 commit into from

Conversation

DanieliMi
Copy link
Contributor

@DanieliMi DanieliMi commented Oct 17, 2018

Currently when searching it is not possible to show the price label. If you perform a search an the autocomplete shows products and their prices these prices are not labeled. Is this behavior intended?

In our case we have many products with tier prices and in the autocomplete will be shown the regular price and the tier price but you cannot distinguish them so we want the "As low as" label to be shown for the tier price. But due to the <span class="regular-price"> it's not displayed by magento2 css.
Basically, prices contained in the template are not always regular prices, they can be other prices as well
so this class "regular-price" does not make sense at this point.

@romainruaud
Copy link
Collaborator

Hello @DanieliMi

for me it's actually unclear what is doing your PR.

Would you mind attaching a screen before/after your modification so that I'm able to understand better what you are trying to achieve here ?

Anyway, thank you for contributing to our module

@DanieliMi
Copy link
Contributor Author

Hey @romainruaud
of course I will explain it better.
Before the PR the tier price of an product is displayed like this:
smile_before
After the PR it's going to look like this:
smile_after

Basically the "As low as" label for tier prices will be displayed again. The problem why it's not displayed lays in the line of code I removed. The product price is wrapped in <span class="regular-price"> even tho logically it's not a regular price it's a tier price and because of that the magento luma theme will not display the label. The styling that is responsible for this can be found here

@romainruaud
Copy link
Collaborator

Ok ! thank you for the explanations.

Seems legit to me, I'll double check on my dev env but I'll probably merge this one.

Thank you for contributing !

@romainruaud
Copy link
Collaborator

OK for me. I just replaced it by #1154 to be based on 2.6.x instead of master.

Thank you for contributing.

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

Successfully merging this pull request may close these issues.

2 participants