Hi Neil, The new margins compile and work in 1.69. Thank you.
I noticed the following: 1) The doc does not state the default type of the new margins (for me, using margin 4, it came up as line numbers which was unexpected - but easy to program around). SetMarginTypeN() says "and the next two" which should be "and the next four" and then the code extended to set this default state. 2) The mouse pointer behaviour is now exactly what I wanted (when I add a "spacer" margin to the left of the text), but the visual effect is wrong as the margin colour is not the same as the background colour of the text, which I had not anticipated... obvious when you think about it. Would you reconsider the original request of allowing the mouse pointer to change when in the left hand margin of the text window and allow line selection when in this region? If people feel they need 1 pixel space (the default) to get the caret to the left of text for swiping it, we could say that this effect only happens when the left margin is > 1 pixel and that the pointer stays as a text caret until more than 1 pixel to the left of the text. If this is not going to be acceptable, then would you consider another margin type code? SetMarginTypeN(int margin, int iType) gets SC_MARGIN_SELECT(3) meaning it is a selection margin and has the text background colour. This feels better than allowing us to set margin background colours, as this would involve adding extra state information to the margins, and extra code in the container to make sure that the margin colour stays in step with the text background. Greg Smith _______________________________________________ Scintilla-interest mailing list [email protected] http://mailman.lyra.org/mailman/listinfo/scintilla-interest
