On October 20th, 2010, 3:41 p.m., Thomas Zander wrote:

The requested feature is already available in the text tool the request to be able to click on links in kpresenters presentation tool should follow the same design if it wants to do the same thing.
Making this into an action sounds wrong to me because that means the functionality becomes available for all tools and that goes against the concept of flake which is that tools are specialized by the functionality that the user wants the work with at this moment.

On October 21st, 2010, 6:45 a.m., Matus Hanzes wrote:

One problem with the TextTool is that I don't want to have IBeamCursor when mouse pointer is hovering over text in text shape.

Another problem is that I want to handle all mouse events in kpresenter when the mouse is over TextShape but not over hyperlink.(go to the next slide) And it is not possible because I can't get the information out of the TextTool.

If you know how to get the information that mouse pointer is over hyperlink out of TextShape. Then tell me.

Sorry if I was not clear; I didn't mean you should use the text tool, I said the presentation tools should use the same design that the text tool already uses.

What I meant with that is because both the presentation tool and the text tool should handle urls, but no other tools should handle URLS those two tools should have code for this behavior.
So please add some code to the presentation tool to check if there is a text shape where the user clicked and then use the public APIs to figure out the URL and handle that accordingly.

How does that sound?

- Thomas

On October 15th, 2010, 2:40 p.m., Matus Hanzes wrote:

Support for hyperlinks in presentation mode consist of two problems
1.	Change the cursor from ArrowCursor to PointingHandCursur, when the cursor is over the hyperlink.
2.	Run the hyperlink, when the hyperlink is clicked.

I have used KoShape EventAction framework for accessing the data inside TextShape.
To work with hyperlinks in TextEvenAction I have used code from TextTool.

I will do unit tests after this solution is approved.