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

Сравнение <noscript> c учетом порядка элементов #136

Open
kompolom opened this issue Feb 13, 2016 · 12 comments
Labels

Comments

@kompolom
Copy link

Diff: bem-site/bem-forum-content-ru#829
Описание: bem-site/bem-forum-content-ru#865

Воспроизводится только на noscript

@eGavr
Copy link
Member

eGavr commented Feb 13, 2016

Отправь реальный пример HTML, пожалуйста.

@kompolom
Copy link
Author

https://github.com/appwilio/ap-components/blob/master/common.blocks/image/image.tmpl-specs/50-semantic-lazy.html Так пойдет?
В качестве шаблонизаторов используются bh.php (4.1) и bh.js

@eGavr
Copy link
Member

eGavr commented Feb 13, 2016

Тулза ничего не знает про шаблонизаторы, они ей неинтересны, она просто сравнивает два HTML. Чтобы воспроизвести багу, мне нужны два HTML, которые по твоему сравниваются неправильно.

@kompolom
Copy link
Author

Ок. как их получить?

@kompolom
Copy link
Author

Я же не генерю их руками. Они создаются enb-tmpl-specs и нигде не сохраняются

@eGavr
Copy link
Member

eGavr commented Feb 13, 2016

@kompolom
Copy link
Author

<span class="image image_load_lazy image_lazy_scroll scrollspy i-bem" role="img" data-bem='{"image":{"url":"/image.jpg"},"scrollspy":{"offset":"-30"}}'><span class="image__container"><img class="image image__img" alt="alt" title="title" role="img"/><span class="image__spin"></span></span>
<noscript
class="image__fallback"><img class="image" src="/image.jpg" alt="alt" role="img" /></noscript>
    </span>

Вот что в desktop.tmpl-specs/image/30-lazy.BH.js.html
Форматирование сохранил.

@eGavr
Copy link
Member

eGavr commented Feb 13, 2016

А второй HTML где ? :) Ты выше уже его кидал? Та забудь ты про шаблонизаторы :)

В этом issue мне от тебя нужно просто два HTML, которые должны считаться равными, но инструмент все равно показывает дифф :)

@kompolom
Copy link
Author

@eGavr ссылку на эталон я скинул выше. В предыдущем коменте второй файл.
фу, сам запутался в этих файлах... Сейчас скину оба вместе..

@kompolom
Copy link
Author

<span class="image image_load_lazy image_lazy_scroll scrollspy i-bem" role="img" data-bem='{"image":{"url":"/image.jpg"},"scrollspy":{"offset":"-30"}}'><span class="image__container"><img class="image image__img" alt="alt" title="title" role="img"/><span class="image__spin"></span></span>
<noscript
class="image__fallback"><img class="image" src="/image.jpg" alt="alt" role="img" /></noscript>
    </span>
<span class="image image_load_lazy image_lazy_scroll scrollspy i-bem"
    role="img"
    data-bem="{&quot;image&quot;:{&quot;url&quot;:&quot;/image.jpg&quot;},&quot;scrollspy&quot;:{&quot;offset&quot;:&quot;-30&quot;}}">
    <span class="image__container">
        <img class="image image__img" role="img" alt="alt" title="title"/>
        <span class="image__spin">
        </span>
    </span>
    <noscript class="image__fallback">
        <img class="image" role="img" src="/image.jpg" alt="alt"/>
    </noscript>
</span>

upd: Diff
2016-02-13 17 21 02

@eGavr
Copy link
Member

eGavr commented Feb 13, 2016

Спасибо, будем смотреть ;)

@eGavr
Copy link
Member

eGavr commented Feb 13, 2016

@kompolom , парсер, который используется в html-differ считает содержимое тега noscript обычным текстом, а не HTML-кодом, поэтому этот участок сравнивается как обычный текст, как следствие получаем дифф.

Завел issue в parse5, можешь присоединяться к обсуждению или просто плюсануть в issue :)

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

No branches or pull requests

2 participants