[Yanel-dev] Using query strings to match [WAS: Re: [HEADS UP] ResourceManager and ResourceConfigurationMap class may die soon; resource-types' URL matching (e.g. on request parameters)
Bettina Messmer
bettina.messmer at gmx.ch
Thu Jul 21 16:52:31 CEST 2011
Hi,
I guess the problem, was
vertical-align: baseline;
after commenting it, the Toolbar worked.
(I added a patch file)
thanks and greets,
Bettina
-------- Original-Nachricht --------
> Datum: Thu, 21 Jul 2011 15:40:52 +0200
> Von: Michael Wechner <michael.wechner at wyona.com>
> An: yanel-development at wyona.com
> Betreff: Re: [Yanel-dev] Using query strings to match [WAS: Re: [HEADS UP] ResourceManager and ResourceConfigurationMap class may die soon; resource-types\' URL matching (e.g. on request parameters)
> Am 21.07.11 15:38, schrieb Bettina Messmer:
> > Hi,
> >
> > As we discussed offline, the problem seems to be that toolbar_v2.css
> does not work with IE9, whereas toolbar.css used by
> "from-scratch-realm"-Toolbar does also work with IE9. We keep investigating, what might be the
> problem of this particular css.
>
> these CSS are not so much different:
>
> diff src/webapp/htdocs/toolbar.css src/webapp/htdocs/toolbar_v2.css
> 0a1,2
> > /* Changes re superfish added (or rather specifc rules removed) */
> >
> 28d29
> < top:0px;
> 35c36
> < top: 3px;
> ---
> > top: 2px;
> 48a50
> > vertical-align: baseline;
> 55a58,60
> > height: 18px;
> > line-height: 18px;
> > background: lightGrey;
> 59a65,67
> > height: 28px;
> > line-height: 28px;
> > vertical-align: baseline;
> 60a69
> > padding-top: 1px;
> 76a86,87
> > margin: 0;
> > padding: 3px;
> 88a100
> > background:lightsteelblue;
> 101a114
> > background:lightsteelblue;
> 117c130
> < padding:2px;
> ---
> > padding:0;
> 125c138,141
> < #yaneltoolbar_menu li .haschild {
> ---
> > #yaneltoolbar_menu li.haschild {
> > height: 18px;
> > line-height: 18px;
> > padding: 3px;
> 131c147,149
> < padding:4px;
> ---
> > padding:0;
> > height: 24px;
> > line-height: 24px;
> 165,176d182
> < div#yaneltoolbar_menu li:hover ul ul,
> < div#yaneltoolbar_menu li li:hover ul ul,
> < div#yaneltoolbar_menu li li li:hover ul ul,
> < div#yaneltoolbar_menu li li li li:hover ul ul
> < {display:none;}
> <
> < div#yaneltoolbar_menu li:hover ul,
> < div#yaneltoolbar_menu li li:hover ul,
> < div#yaneltoolbar_menu li li li:hover ul,
> < div#yaneltoolbar_menu li li li li:hover ul
> < {display:block;}
>
> Thanks for investigating
>
> Michael
> > greets,
> >
> > Bettina
> >
> >
> >
> >
> > -------- Original-Nachricht --------
> >> Datum: Wed, 20 Jul 2011 15:26:11 +0200
> >> Von: Michael Wechner<michael.wechner at wyona.com>
> >> An: yanel-development at wyona.com
> >> Betreff: [Yanel-dev] Using query strings to match [WAS: Re: [HEADS UP]
> ResourceManager and ResourceConfigurationMap class may die soon;
> resource-types\' URL matching (e.g. on request parameters)
> >> Hi
> >>
> >> Am 18.07.11 14:41, schrieb Michael Wechner:
> >>> Hi
> >>>
> >>> I have implemented this patch of Guillaume, whereas did some
> >>> modification to it.
> >>>
> >>> The next steps are:
> >>>
> >>> - Develop another matcher implementation which allows to use query
> >>> strings for mapping
> >> I have developed another matcher which allows now to use query strings
> >>
> >> src/core/java/org/wyona/yanel/core/ResourceConfigurationMapV2.java
> >>
> >> It's not perfect yet, but for playing it should work fine. One
> currently
> >> has to enable it by changing:
> >>
> >> Index: src/core/java/org/wyona/yanel/core/ResourceManager.java
> >> ===================================================================
> >> --- src/core/java/org/wyona/yanel/core/ResourceManager.java
> (revision
> >> 59475)
> >> +++ src/core/java/org/wyona/yanel/core/ResourceManager.java (working
> >> copy)
> >> @@ -47,8 +47,8 @@
> >> */
> >> public ResourceManager() {
> >> rtRegistry = new ResourceTypeRegistry();
> >> - resourceTypeMatcher = new ResourceTypeDefaultMatcher();
> >> - // TODO: Make resource matcher configurable:
> >> resourceTypeMatcher = new ResourceTypeMatcherV1ImplV2();
> >> + //resourceTypeMatcher = new ResourceTypeDefaultMatcher();
> >> + resourceTypeMatcher = new ResourceTypeMatcherV1ImplV2();
> >> }
> >>
> >> One can dow now stuff like for example:
> >>
> >> <matcher pattern="/**.jpg?scale=mobile"
> >> rcpath="/scale-image_yanel-rc.xml"/>
> >>
> >> whereas for requests/URLs containing query strings, but the pattern
> >> contains no question mark the matching
> >> should still work, which means it is backwards compatible.
> >>
> >> We need to do more testing before enabling this new version for good
> (or
> >> as mentioning below, make it at least configurable, such that one can
> >> easily switch back).
> >>
> >> Also we need to discuss about POST parameters, etc. Also see previous
> >> threads:
> >>
> >>
> http://lists.wyona.org/pipermail/yanel-development/2009-December/004235.html
> >>
> http://lists.wyona.org/pipermail/yanel-development/2009-December/004194.html
> >>
> >> Also we can think about further re-factoring of
> >>
> >>
> http://lists.wyona.org/pipermail/yanel-development/2009-July/003844.html
> >>
> >> whereas we quickly get into discussions re access control as well ;-)
> >>
> >> Thanks
> >>
> >> Michael
> >>
> >>> - Make matcher configurable per yanel and per realm
> >>>
> >> --
> >> Yanel-development mailing list Yanel-development at wyona.com
> >> http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development
>
> --
> Yanel-development mailing list Yanel-development at wyona.com
> http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development
--
NEU: FreePhone - 0ct/min Handyspartarif mit Geld-zurück-Garantie!
Jetzt informieren: http://www.gmx.net/de/go/freephone
-------------- next part --------------
A non-text attachment was scrubbed...
Name: toolbar_v2_2.patch
Type: application/octet-stream
Size: 1069 bytes
Desc: not available
URL: <http://lists.wyona.org/pipermail/yanel-development/attachments/20110721/6e8f88e8/attachment.obj>
More information about the Yanel-development
mailing list