myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simon Lessard" <simon.lessar...@gmail.com>
Subject Re: svn commit: r576576 [1/3] - in /myfaces/trinidad/trunk/trinidad: trinidad-build/src/main/resources/META-INF/maven-faces-plugin/components/trinidad/core/ trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/ trinida
Date Tue, 18 Sep 2007 19:24:32 GMT
That's the div shown when there's a visible dialog I think, because
everything still works during PPR events.

On 9/18/07, Andrew Robinson <andrew.rw.robinson@gmail.com> wrote:
>
> There is already a blocking panel rendered by Trinidad. Looks like this:
>
> <div id="_pprBlockingDiv" onkeypress="return false;" onmouseup="return
> false;" onmousedown="return false;" onkeyup="return false;"
> onkeydown="return false;" style="position: absolute; left: 0pt; top:
> 0pt; width: 0pt; height: 0pt; cursor: wait;" onclick="return
> _pprConsumeClick(event);"/>
>
> Not sure if you would want a component that simply shows this or not,
> I would think a public javascript API method may be a more obvious
> choice?
>
> On 9/18/07, Simon Lessard <simon.lessard.3@gmail.com> wrote:
> > Another way would be a "blocking panel" component that you set in the
> > statusIndicator's busy facet. Blocking panel skinning could be set to
> > absolute positioning to cover the whole screen and apply transparency
> > masking. Some JavaScript could also be added to it to prevent event
> > bubbling. That would effectively block everything during PPR events.
> >
> > That being said, you still didn't give me your input on what should be
> > rendered when only one facet is specified. ;P
> >
> >
> > On 9/18/07, Andrew Robinson <andrew.rw.robinson@gmail.com> wrote:
> > > There is no reason to change the cursor unless you want the GUI to be
> > > unresponsive to clicks and key presses right? The goal of a busy
> > > cursor would be to block any user input and let them know things are
> > > busy. In that case this works:
> > >
> > > <tr:commandLink partialSubmit="true" blocking="true" />
> > >
> > > Then if that is indeed what is desired, then maybe what is best is to
> > > have all components that have "autoSubmit" support also support
> > > "blocking".
> > >
> > > Along with that, perhaps the ability for blocking should also be added
> > > to the JavaScript call "TrPage.getInstance().sendPartialFormPost"
> > >
> > > I would see this as a more flexible architecture, as it leaves the
> > > choice of whether or not to block user input up to each control
> > > instead of making the blocking a choice of the status indicator.
> > >
> > > Different approaches could be (1) always block on PPR or (2) block on
> > > certain types of PPR only or (3) block on the majority PPR but with
> > > the occasional exception (like a poll for example).
> > >
> > > Just my $.02
> > >
> > > -Andrew
> > >
> > > On 9/18/07, Simon Lessard <simon.lessard.3@gmail.com> wrote:
> > > > A way to set the cursor directly would be nice. I'm sure we're going
> to
> > see
> > > > users wanting that. Is there a way to directly set the current
> cursor?
> > > > Dunno, some obscure JavaScript function...
> > > >
> > > > Also, I could use more input on the indicator behavior in case only
> one
> > > > facet is specified and/or rendered.
> > > >
> > > >
> > > > On 9/18/07, Matthias Wessendorf <matzew@apache.org> wrote:
> > > > > ...was just a thought ... :-)
> > > > >
> > > > > On 9/18/07, Andrew Robinson <andrew.rw.robinson@gmail.com>
wrote:
> > > > > > BTW: this does work, but is ugly:
> > > > > >
> > > > > > BODY * {
> > > > > >   cursor: pointer !important;
> > > > > > }
> > > > > >
> > > > > >
> > > > > > On 9/18/07, Andrew Robinson <andrew.rw.robinson@gmail.com
>
> wrote:
> > > > > > > That does not fully work, in Firefox at least (just tried
it,
> the
> > > > > > > cursor still reverts to an I-Beam for input text boxes
for
> > example)
> > > > > > >
> > > > > > > On 9/18/07, Simon Lessard <simon.lessard.3@gmail.com>
wrote:
> > > > > > > > It's possible to set the cursor on the body node,
would have
> to
> > use
> > > > a skin
> > > > > > > > property rather than a style property, but I'm not
sure I
> like
> > that.
> > > > > > > >
> > > > > > > >
> > > > > > > > On 9/18/07, Andrew Robinson < andrew.rw.robinson@gmail.com>
> > wrote:
> > > > > > > > > I'm not sure a cursor will work. The cursor applies
to the
> > > > front-most
> > > > > > > > > element underneath the cursor. The only way,
that I am
> aware
> > of,
> > > > to
> > > > > > > > > get a global cursor is to float (using absolute
> positioning
> > and
> > > > > > > > > z-index) an HTML element (like a DIV) over the
entire
> page.
> > This
> > > > is
> > > > > > > > > what the "blocking" functionality already does
in Trinidad
> (I
> > > > > > > > > believe).
> > > > > > > > >
> > > > > > > > > So really, is it not the job of the programmer
to ensure
> the
> > PPR
> > > > > > > > > request is a blocking request and not the job
of the
> status
> > > > indicator
> > > > > > > > > to change the cursor?
> > > > > > > > >
> > > > > > > > > On 9/18/07, Simon Lessard < simon.lessard.3@gmail.com>
> wrote:
> > > > > > > > > > Hmm I don't think this should be directly
in the
> indicator
> > > > contract. If
> > > > > > > > it
> > > > > > > > > > is, then I think it should be a new "type"
attribute on
> the
> > > > status
> > > > > > > > indicator
> > > > > > > > > > with the following values:
> > > > > > > > > >
> > > > > > > > > > "icon": render default icons (default value)
> > > > > > > > > > "cursor": change the cursor on document
level (requires
> a
> > way to
> > > > > > > > specified
> > > > > > > > > > the busy and ready cursor though)
> > > > > > > > > > "facets": render busy and ready facets
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > On 9/18/07, Matthias Wessendorf <matzew@apache.org>
> wrote:
> > > > > > > > > > > one more,
> > > > > > > > > > >
> > > > > > > > > > > what about changing the cursor, when
statusIndicator
> is
> > busy ?
> > > > > > > > > > >
> > > > > > > > > > > -M
> > > > > > > > > > >
> > > > > > > > > > > On 9/18/07, Matthias Wessendorf <
matzew@apache.org >
> > wrote:
> > > > > > > > > > > > I think
> > > > > > > > > > > >
> > > > > > > > > > > > no facet => icon is rendered;
> > > > > > > > > > > > otherwise, no icon is rendered.
> > > > > > > > > > > >
> > > > > > > > > > > > -M
> > > > > > > > > > > >
> > > > > > > > > > > > On 9/18/07, Simon Lessard <
> simon.lessard.3@gmail.com>
> > > > wrote:
> > > > > > > > > > > > > Hmm not as simple as I though.
Before pushing a
> patch
> > let
> > > > decide
> > > > > > > > on
> > > > > > > > > > the
> > > > > > > > > > > > > behavior for every use case:
> > > > > > > > > > > > >
> > > > > > > > > > > > > Both facets are specified
and rendered --> Don't
> > render
> > > > any icon
> > > > > > > > > > > > > Both facets are specified
but only one is rendered
> -->
> > ?
> > > > > > > > > > > > >  Both facets are specified
but neither are
> rendered
> > --> ?
> > > > > > > > > > > > >  Only one facet is specified
and rendered -->
> Don't
> > render
> > > > any
> > > > > > > > icon or
> > > > > > > > > > > > > render the icon of the missing
facet?
> > > > > > > > > > > > > Only one facet is specified
but not rendered --> ?
> > > > > > > > > > > > > No facet is specified -->
Render both icons
> > > > > > > > > > > > >
> > > > > > > > > > > > > ~ Simon
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > > On 9/18/07, Simon Lessard
<
> simon.lessard.3@gmail.com>
> > > > wrote:
> > > > > > > > > > > > > > Or put tr:icon in the
facet. Yeah, that sound
> good
> > too.
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > On 9/18/07, Matthias
Wessendorf <
> matzew@apache.org
> > >
> > > > wrote:
> > > > > > > > > > > > > > > that sounds like
the best solution.
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > On 9/18/07, Adam
Winer < awiner@gmail.com>
> wrote:
> > > > > > > > > > > > > > > > IMO, if we
have a facet, we don't render the
> > icon.
> > > > No need
> > > > > > > > > > > > > > > > for an attribute
at all.
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > Anyone that
desperately needs both the facet
> and
> > the
> > > > icon
> > > > > > > > > > > > > > > > can render
two statusIndicators.
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > -- Adam
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > On 9/18/07,
Matthias Wessendorf
> > <matzew@apache.org >
> > > > wrote:
> > > > > > > > > > > > > > > > > Hi,
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > On 9/18/07,
Simon Lessard <
> > > > simon.lessard.3@gmail.com >
> > > > > > > > wrote:
> > > > > > > > > > > > > > > > > >
Speaking of which, I forgot to add skin
> > > > documentation.
> > > > > > > > I'll
> > > > > > > > > > do
> > > > > > > > > > > > > that right
> > > > > > > > > > > > > > > > > >
away.
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > >
I would also like to add a new attribute
> to
> > skip
> > > > the
> > > > > > > > icon
> > > > > > > > > > > > > rendering. If it
> > > > > > > > > > > > > > > > > >
hasn't been of backward compatibility, I
> > would
> > > > have
> > > > > > > > simply
> > > > > > > > > > removed
> > > > > > > > > > > > > them
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > I added
a demo usage of the facet's, I was
> > > > thinking, that
> > > > > > > > it
> > > > > > > > > > > > > shouldn't
> > > > > > > > > > > > > > > > > render
the "default" icon,
> > > > > > > > > > > > > > > > > glad
you pointed it out now.
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > >
since it's easily doable with a
> combination
> > of
> > > > facet and
> > > > > > > > > > tr:icon,
> > > > > > > > > > > > > but since
> > > > > > > > > > > > > > > > > >
we had a release with the
> statusIndicator
> > > > already,
> > > > > > > > that's
> > > > > > > > > > out of
> > > > > > > > > > > > > question.
> > > > > > > > > > > > > > > > > >
So, what I need now is a decent
> attribute
> > name.
> > > > What do
> > > > > > > > you
> > > > > > > > > > think
> > > > > > > > > > > > > of
> > > > > > > > > > > > > > > > > >
"renderIcon" or "renderFacetsOnly"?
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > I tend
to like renderFacetsOnly, because
> that
> > what
> > > > you
> > > > > > > > added
> > > > > > > > > > where
> > > > > > > > > > > > > facets.
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > Perhaps,
we can change that soon, that
> when
> > > > facet's are
> > > > > > > > > > specified,
> > > > > > > > > > > > > we
> > > > > > > > > > > > > > > > > don't
render the "default" icon.
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > -Matthias
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > >
~ Simon
> > > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > > > Matthias
Wessendorf
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > > further
stuff:
> > > > > > > > > > > > > > > > > blog:
> > > > > > > > > > http://matthiaswessendorf.wordpress.com/
> > > > > > > > > > > > > > > > > mail:
matzew-at-apache-dot-org
> > > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > Matthias Wessendorf
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > further stuff:
> > > > > > > > > > > > > > > blog:
> > > > > > > > http://matthiaswessendorf.wordpress.com/
> > > > > > > > > > > > > > > mail: matzew-at-apache-dot-org
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > > --
> > > > > > > > > > > > Matthias Wessendorf
> > > > > > > > > > > >
> > > > > > > > > > > > further stuff:
> > > > > > > > > > > > blog:
> > > > http://matthiaswessendorf.wordpress.com/
> > > > > > > > > > > > mail: matzew-at-apache-dot-org
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > --
> > > > > > > > > > > Matthias Wessendorf
> > > > > > > > > > >
> > > > > > > > > > > further stuff:
> > > > > > > > > > > blog:
> > > > http://matthiaswessendorf.wordpress.com/
> > > > > > > > > > > mail: matzew-at-apache-dot-org
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Matthias Wessendorf
> > > > >
> > > > > further stuff:
> > > > > blog: http://matthiaswessendorf.wordpress.com/
> > > > > mail: matzew-at-apache-dot-org
> > > > >
> > > >
> > > >
> > >
> >
> >
>

Mime
View raw message