Mastercard / flow

Testing framework

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Improve message-web text format

therealryan opened this issue · comments

message-web's treatment of multiline values leaves a little something to be desired - it's good that it indents them, but in the result (for example) it's difficult to spot where the padding ends and the content begins.

We should improve the output to make it more obvious (perhaps?).

Hi, you are right. I will have a look into this. Also this is my first issue so I am kinda excited to help :D

Cheers

Btw, what output do you exactly suggest?

I think it'd probably be enough to put characters at the right-hand side of the padding. That should make it more obvious where the content begins

If you were feeling especially fancy, you could put a on the last line of multiline content to make the distinction from the next value clearer

Give me a shout if you need any help getting the project set up and building. As you won't be changing the angular app you can avoid having to install node if you want (just build with, e.g.: mvn test -Dnode=none.
This test will show the impact of your changes.