-
Notifications
You must be signed in to change notification settings - Fork 45
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
Comments
👍 |
Я тут подумал над это фичей... нужна ли она? Кроме этого, в дифф может попасть не только сама маска, но и еще кусок обычного текста, например, в тех случаях, когда маска является частью слова: |
|
@zxqfox , бывает ^_^ |
@eGavr мой посыл в том, что маски сравниваются иначе, чем любые другие символы или слова. И из классического двухцветного диффа, я бы не понял, что есть какая-то магия в сравнении. Может быть сделаем опционально? А сейчас нет возможности показывать побуквенный дифф? Чтобы |
Нет, его и не будет, так как при большом диффе в логе будет АД. (есть таска про создание опции, но даже если ее реализовать, то мы не можем в зависимости от входных данных ее настраивать, это неправильно с точки зрения использования опции) А мой посыл в том, что если человек заюзал маски намеренно, то он и понимает, что эти самые маски делают. |
Почему? Посмотри на дифф гитхаба, он как раз посимвольный, но немного хитрее. Есть причины не сделать так же? |
А зачем столько цветов, они будут мешать. |
И гитхабу лог делать удобнее, так как он не работает с инпутом, который представляется в виде одной строки. |
Ну подождите, кто мешает строить некое AST для хтмл (по тегам) и сранивать отдельно само дерево, затем его содержимое, и затем собирать вывод. |
Раньше HTML преобразовывался в AST, но это решение через AST не давала нужного результата, потому что нужно получать инфу по входным данных AS IS. |
For example, you can use
bgMagenta
color.The text was updated successfully, but these errors were encountered: