Gisellameloni / svg-edit

Automatically exported from code.google.com/p/svg-edit
MIT License
0 stars 0 forks source link

Text tool cursor keeps flashing in the same place after resizing text #1134

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Choose the text tool and then click on the canvas
2. Type anything e.g. "Hello" but do not de-select
3. Use the visible resize handles to make the text bigger
4. Move and then de-select the text element

What is the expected output? What do you see instead?
I expect to not see any text cursor since I am no longer editing text, but 
instead I see the cursor lingering in its original location, still flashing: 
http://d.pr/i/6v7

In what browser did you experience this problem? (ALL, Firefox, Opera, etc)
Chrome 29.0.1547.57, Firefox 23.0.1, and Safari 5.1.9 on OS X 10.6.8

In what version of SVG-edit does the problem occur? (Latest trunk, 2.5.1,
etc)
Latest trunk

Please provide any additional information below.
I've only been able to reproduce this error while initially creating a new text 
element. This is because if I try to edit an existing text element, I don't see 
the resize handles until I de-select the text and then the cursor properly goes 
away.

At a high level, we could either fix the lingering cursor bug with the existing 
functionality OR disable the resize handles upon creation of a new text 
element, while it's in "edit mode", just like for editing existing text 
elements.

Original issue reported on code.google.com by dan...@trydesignlab.com on 28 Aug 2013 at 10:49

GoogleCodeExporter commented 8 years ago
After lots of digging through the svg-edit code, I came up with this quick fix: 
https://github.com/danxshap/svg-edit/pull/2

Thoughts?

Original comment by dan...@trydesignlab.com on 29 Aug 2013 at 1:02

GoogleCodeExporter commented 8 years ago

Original comment by dan...@trydesignlab.com on 18 Sep 2013 at 11:36

GoogleCodeExporter commented 8 years ago
A simpler case is
1. Choose the text tool and then click on the canvas
2. Use the visible resize handles to make the text bigger
3. De-select the text element

Original comment by rockyk...@gmail.com on 16 Oct 2014 at 4:50

GoogleCodeExporter commented 8 years ago
yep. if you make the change that i suggested, this issue will go away for you.

Original comment by dan...@trydesignlab.com on 16 Oct 2014 at 2:06