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).
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).