Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Scott Petrovic
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120505/#review68246 --- I made a couple comments on the forum about improving the UI

Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Sven Langkamp
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120505/#review68238 --- krita/plugins/assistants/RulerAssistant/PerspectiveAssistant.

Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Wolthera van Hövell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120505/ --- (Updated Oct. 10, 2014, 9:32 p.m.) Review request for Calligra, Dmitry Ka

Re: Review Request 120547: krita: save text tool options

2014-10-10 Thread Sven Langkamp
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120547/#review68237 --- krita/plugins/tools/tool_text/kis_tool_text.cc

Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Wolthera van Hövell
> On Oct. 10, 2014, 8:13 p.m., Sven Langkamp wrote: > > KisPaintingAssistantsDecoration is a KisCanvasDecoration which already has > > setVisible and toggleVisible. It's not needed to do the > > setAssistantVisible, I think. Yes indeed, it does. However, if we use that, it won't allow me to pa

Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Sven Langkamp
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120505/#review68228 --- KisPaintingAssistantsDecoration is a KisCanvasDecoration which

Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Wolthera van Hövell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120505/ --- (Updated Oct. 10, 2014, 7:06 p.m.) Review request for Calligra, Dmitry Ka

Some questions about Calligra Braindump

2014-10-10 Thread Joshua J. Kugler
Since there is no user list, frinring in #Calligra said I could post here. :) This is Braindump 0.10.9 on KDE 4.13.2 1. When I create a text box, and then try to do connectors, no connectors show up on the outline of the text shape. However, if I create a connector from a "real" shape (such as

Re: Review Request 120477: Unbreak build without Glew

2014-10-10 Thread Friedrich W. H. Kossebau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120477/ --- (Updated Oct. 10, 2014, 2:28 p.m.) Status -- This change has been ma

Re: Review Request 120477: Unbreak build without Glew

2014-10-10 Thread Boudewijn Rempt
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120477/#review68210 --- Ship it! Ship It! - Boudewijn Rempt On Oct. 3, 2014, 2:07

Re: Review Request 120291: cleaning up Advanced Colour Selector settings+Configurable HSY' luma.

2014-10-10 Thread Wolthera van Hövell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120291/ --- (Updated Oct. 10, 2014, 2:09 p.m.) Status -- This change has been ma

Re: Review Request 120505: Adding a activation-toggle and preview to assistants.

2014-10-10 Thread Wolthera van Hövell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120505/ --- (Updated Oct. 10, 2014, 1:13 p.m.) Status -- This change has been ma