Hi,
On Wed, Dec 16, 2009 at 9:48 AM, Thomas Mortagne
<thomas.mortagne(a)xwiki.com>wrote;wrote:
On Wed, Dec 16, 2009 at 08:30, Marius Dumitru
Florea
<mariusdumitru.florea(a)xwiki.com> wrote:
> Hi devs,
>
> Currently we have this behavior:
>
> * simple click to select a macro
> * simple click to toggle between collapsed and expanded state of a
> previously selected macro
> * double click to edit the macro properties
>
> The problem is that the double click event consists, as its name
> suggests, in two consecutive click events. As a consequence, when you
> double click to edit a macro you also toggle its visibility state.
> Besides being annoying, this can lead sometimes to an unexpected result:
>
> -----<details> -----
> I inserted a really long code macro (paste an entire Java source file).
> and them selected the macro and double clicked somewhere in the middle
> to edit its properties. This is what happened:
>
> * The first click event collapsed the macro
> * The second click event was not fired on the macro but on document
> body, because after the macro collapsed the place where I clicked was in
> the middle of nowhere. As a result the macro was unselected.
> * the (logical) double click event was still fired on the macro (I guess
> because the target of the first click was the macro container) and thus
> the edit macro dialog opened
> * I changed the macro properties and closed the dialog. As a result the
> macro was duplicated. The edit dialog should have replaced the selected
> macro but there was no selected macro..
> -----</details> -----
>
> Therefore I propose to use a modifier key with click to collapse/expand
> a macro. I'm not sure which modifier key is the best. I think we should
> choose between Alt and Meta. The behavior would become:
>
> * simple click to select a macro
> * [Alt or Meta] + simple click to toggle the collapsed and expanded state
> * double click to edit
>
> I'm +1 for Alt key.
>
> WDYT?
What about having to click in a specific area like a +/- in the top
left corner to toggle the collapsed and expanded state ? This sounds
a lot easier and natural for a user, most of users will never remember
how to do it if it's " [Alt or Meta] + simple click".
Yes, that's what I was thinking about too. A "minimize" button at
the top
right of the macro displayer would be better.
IMO macros should look as in view
mode when expanded. Right now this is
not happening all the time due to the current implementation which uses
buttons to protect the macro output and buttons have an inherent
inline-block display. But in the future, when the contentEditable
attribute will be fully supported, we should have:
before |this is a multiple line macro output blah blah
blah blah blah blah blah| after
instead of
before |this is a multiple | after
|line macro output |
|blah blah blah blah|
|blah blah blah |
Thus putting a +/- sign in the top left corner seems artificial to me,
considering that the macro output is not necessarily a box.
Coming back to the current implementation, I don't think it's possible
to have a button inside another button and to catch the click event for
the inner button because you can't actually click the inner button
without clicking the outer button. If that would work then you would be
able to interact with the macro output in edit mode (e.g. click links,
move to the next page in a live table, etc.), which doesn't happen right
now.