Greg Smith:

1) The doc does not state the default type of the new margins

  Its undefined.

program around). SetMarginTypeN() says "and the next two" which should be
"and the next four"

  There is no convention for the new margins as SciTE and other
editors don't use them as yet.

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?

  This complicates the role and features of the left margin.

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.

  Don't call it a selection margin type as that implies the other
margin types shouldn't be capable of selection.

  Neil
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest

Reply via email to