Snap tooltip not fully drawn

Bug #336402 reported by Diederik van Lierop
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Low
Diederik van Lierop

Bug Description

Using rev. #20792 on Linux

After snapping, a tooltip is being drawn. In some cases this tooltip is only partly drawn or not drawn at all, see the attached screenshot. This screenshot has been made with DEBUG_REDRAW enabled (see line 62 in sp-canvas.cpp). The yellow rectangles are for debugging only and show which parts of the canvas are being redrawn. The tooltip spans across multiple of these rectangles, and is drawn in one rectangle but not the other.

When I replace

     sp_canvas_request_redraw (item->canvas, (int)item->x1, (int)item->y1, (int)item->x2, (int)item->y2);

by

     sp_canvas_request_redraw (item->canvas, -10000, -10000, 10000, 10000);

in sp_canvastext_update() in display/canvas-bpath.cpp then everything looks fine. That's because it causes the full canvas to be redrawn. This doesn't necessarily mean though that this line is at fault.

I tried hunting this bug down, but I don't have a clue what's going wrong here.

PS1: Please note that we currently have 2 copies of sp_canvastext_update, one in canvas-text.cpp and one in canvas-bpath.cpp. Only the latter is being used
PS2: I've attached a sample file, if that's of any use.

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :
Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :
description: updated
Revision history for this message
bbyak (buliabyak) wrote :

i cannot repro in 20883, can you still repro this? if so can you give detailed steps - what to snap to what in what tool etc.

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote : Re: [Bug 336402] Re: Snap tooltip not fully drawn

Unfortunately I can still reproduce this quite easily:
1) Start a new document, and maximize Inkscape
2) Zoom out a bit, such that the page fills less than half the height of
the canvas
3) Create a vertical guideline along the middle of the paper
4) Use for example the bezier tool, and draw a line from the lower left
corner of the canvas and try snap the other end to the guideline

Now when you move the end point along the vertical guideline, you should
see that the snap tooltip is not correctly drawn, but only when you try
snapping to a part of the guideline that lies outside the paper borders.

Revision history for this message
jazzynico (jazzynico) wrote :

Confirmed on Ubuntu 9.04, Inkscape rev. 21025.

Changed in inkscape:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
jazzynico (jazzynico) wrote :

... rev. 22025.
See attachment.

tags: added: snapping ui
tags: added: renderer
Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :

I can't reproduce this any longer

Revision history for this message
su_v (suv-lp) wrote :

Diederik van Lierop wrote on 2011-02-22:
> I can't reproduce this any longer

Still reproduced with current trunk r10065 on OS X 10.5.8 (i386)
(see attached screenshot)

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :

Also happens with the new measurement tool, see attached screenshot

Revision history for this message
Diederik van Lierop (mail-diedenrezi) wrote :

It looks like this has been fixed accidentally and finally ;-). I've claimed before that I couldn't reproduce it, so is it really true?

Please test rev. #10816 or newer

Revision history for this message
su_v (suv-lp) wrote :

Diederik van Lierop wrote:
> I've claimed before that I couldn't reproduce it, so is it really true?

Can't reproduce either anymore - let's close this as 'Fix Committed' for now (and reopen in case someone manages to reproduce with current trunk - latest revision I tested is r12170).

Changed in inkscape:
assignee: nobody → Diederik van Lierop (mail-diedenrezi)
milestone: none → 0.49
status: Confirmed → Fix Committed
Bryce Harrington (bryce)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.