Issue 284: TortoiseMerge Accelerator Key & Menu Reorganisation

Stefan,

I've started trying to put a patch together for this issue. It all
seems quite straightforward apart from one thing, the text block to use.

The current behaviour is for a right click to bring up a menu. The
position of the right click determines the text block to use. In order
to use accelerator keys you need to determine the current text block
cursor position. At present there is no single cursor. Next/Prev
Difference use m_nDiffStart/End as cursors but somewhat confusingly both
Next/Prev Conflicts and searching use the m_nSelBlockStart/End cursors.

Should all three functions (Search, differences, conflicts) use the same
cursor or should we introduce a search cursor and a view cursor to go
alongside the difference and conflict cursors? Or have I missed
something when looking at the code?

Regards

Jeremy Whitby

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Sun Sep 10 12:32:18 2006