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

Highlight masks with different color #124

Open
blond opened this issue Feb 20, 2015 · 11 comments
Open

Highlight masks with different color #124

blond opened this issue Feb 20, 2015 · 11 comments

Comments

@blond
Copy link
Member

blond commented Feb 20, 2015

For example, you can use bgMagenta color.

@eGavr
Copy link
Member

eGavr commented Feb 20, 2015

👍

@eGavr
Copy link
Member

eGavr commented Feb 26, 2015

Я тут подумал над это фичей... нужна ли она?
Кажется, добавление еще одного цвета сделает лог избыточным.
Пользователь, который использует маски должен понимать, что в логе в качестве actual или expected может выступать маска. И эта маска должна подсвечиваться как и любой другой дифф, потому что это и есть дифф по одному из критериев сравнения.
Поэтому предлагаю не путать пользователя в цветах лога и оставить все как есть.

Кроме этого, в дифф может попасть не только сама маска, но и еще кусок обычного текста, например, в тех случаях, когда маска является частью слова:

2015-02-26 20 04 11

@qfox
Copy link
Member

qfox commented Feb 26, 2015

Это нужно тем, у кого терминал светлый, например. Лучше через ENV.
Ой, не тот issue.

@eGavr
Copy link
Member

eGavr commented Feb 26, 2015

@zxqfox , бывает ^_^

@blond
Copy link
Member Author

blond commented Feb 26, 2015

@eGavr мой посыл в том, что маски сравниваются иначе, чем любые другие символы или слова. И из классического двухцветного диффа, я бы не понял, что есть какая-то магия в сравнении.

Может быть сделаем опционально?

А сейчас нет возможности показывать побуквенный дифф? Чтобы textS vs text{{\d}} показало только S vs {{\d}}? Тогда можно было бы подсвечивать только символы маски по особенному.

@eGavr
Copy link
Member

eGavr commented Feb 26, 2015

А сейчас нет возможности показывать побуквенный дифф?

Нет, его и не будет, так как при большом диффе в логе будет АД. (есть таска про создание опции, но даже если ее реализовать, то мы не можем в зависимости от входных данных ее настраивать, это неправильно с точки зрения использования опции)

А мой посыл в том, что если человек заюзал маски намеренно, то он и понимает, что эти самые маски делают.
Главное, что есть дифф, дифф длинной в слово и из него нет никакого труда понять что к чему.
+
@andrewblond , нам это не использовать ^^, надо ждать жалоб от пользователей, но не думаю, что эти жалобы будут.

@blond
Copy link
Member Author

blond commented Feb 26, 2015

Нет, его и не будет, так как при большом диффе в логе будет АД.

Почему? Посмотри на дифф гитхаба, он как раз посимвольный, но немного хитрее. Есть причины не сделать так же?

@eGavr
Copy link
Member

eGavr commented Feb 26, 2015

А зачем столько цветов, они будут мешать.
Хочешь кинуть ПР гитхабового диффа?)

@eGavr
Copy link
Member

eGavr commented Feb 26, 2015

И гитхабу лог делать удобнее, так как он не работает с инпутом, который представляется в виде одной строки.

@qfox
Copy link
Member

qfox commented Feb 26, 2015

Ну подождите, кто мешает строить некое AST для хтмл (по тегам) и сранивать отдельно само дерево, затем его содержимое, и затем собирать вывод.

@eGavr
Copy link
Member

eGavr commented Feb 26, 2015

Раньше HTML преобразовывался в AST, но это решение через AST не давала нужного результата, потому что нужно получать инфу по входным данных AS IS.
В общем, в html-differ используется SAX парсер

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

No branches or pull requests

3 participants