Comment 10 for bug 1845460

Revision history for this message
Jeff Young (jeyjey) wrote :

Hi Fabien,

1) Any reason Find Marker was removed? Is it extraneous?
2) I'd rather see the dialog get a bit wider than lose the "Match" in front of "Case".
3) I'd also be inclined to put the "Include" options on 3 separate lines so that they look visually distinct from the match options.

I gave the code a quick once-over and it looks good. I'll be out tomorrow but hopefully I'll get a chance to build and test it on Thurs.

Cheers,
Jeff.