Fixes a less compilation error: '.no-link a' isn't defined when .emai… #22332
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…l-non-inline() isn't active, so when using '.no-link a', it should be wrapped inside .email-non-inline().
Description (*)
This was discovered by using a different less compiler then what Magento ships with.
The selector
.no-link a
only exists when.email-non-inline()
is active, see:https://github.com/magento/magento2/blob/2.3.1/app/design/frontend/Magento/blank/web/css/source/_email-base.less#L136-L140
However, it gets used in the Magento/blank & Magento/luma themes without being wrapped in
.email-non-inline()
.This throws an error when using less.js compiler v2.7.3 when compiling the
email-inline.less
file. Error message:extend ' .no-link a' has no matches
.See #22330 for more details.
Fixed Issues (if relevant)
Manual testing scenarios (*)
npm install less@2.7.3
bin/magento setup:static-content:deploy -f -t Magento/luma en_US
node_modules/.bin/lessc --no-color var/view_preprocessed/pub/static/frontend/Magento/luma/en_US/css/email-inline.less | head
extend ' .no-link a' has no matches
, with this fix that error is gone.#333333
, not in#006bb4
.Contribution checklist (*)