Comment 5 for bug 941597

Revision history for this message
zcgucas (zcgucas) wrote :

Hi, Johan, in fact the code still checks whether _font is null after those lines I changed. In the member function _pickItem, the original line "if (!_font || !_bbox) return NULL;" is replaced by "if (!_font || !_bbox || (!_drawing.outline() && invisible) ) return NULL;". We could see that it still checks whether _font is NULL. The added code is to ensure texts are not selectable when it is not in outline mode when they are set to be invisible(This is what inkscape does before I changed any code).