On 2011-05-05 13:21, Hans-Emil Skogh wrote
>> I thought about it. It is quite hard question even it does not looks like.
>> There is split line you can move. You are saying that when you make left
>> view narrow and right wide the small one wins (in term of number of
>> chars per line)?
> Hmm. Good question. I would spontaneously suggest to disable the splitter and fix it at 50% when "Wrap long lines" is enabled.
I have started feature request as you have seen
> Another suggestion would be to just make the whole line-wrapped block behave as a single block when it comes to the diff. That is, show the all the lines that makes up the wrapped lines as changed, if any part of the lines is changed. Not as useful or elegant, but it's a possibility...
Sorry don't get what you mean ... Is it not like today ?
>>> In addition, I think it would be good if TortoiseMerge could try to
>>> make the visualization of broken lines more readable by considering
>>> inline difference information (that we already have) when deciding
>>> where to break the line.
>> Sounds good, however if you add one char - that char should take line?
> No. I think that there should be a minimum length requirement. Perhaps 50% of the line width or so.
>> or same space should be inserted into other view?
> Yes, if it's a new "line" I think there should be a space at the opposite line.
>> If second there are different char sizes (KANJI) and same differs in
>> size by content nearby (FARSI), so that easy, and readability of such
>> expanded text would be heavy affected.
> How are those cases handled in the inline diff at the bottom of the TortoiseMerge window? I think we should see the same problems there as well, no?
Kind of, let say you rename VeryLongVariableName into i then every
ocurance of this diff shift text in one line. So after few changes lines
are in offset even more then 100 chars, effectively looking like you
look on two totaly differnent lines ...
However don't see any real way how to make it better while keep readability.
Oto BREZINA, Printflow s.r.o., EU
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-05-05 15:14:47 CEST