[OFBIZ-3442] Replace popup lookups by layer lookups

• Type: Sub-task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: Trunk
• Component/s: ALL APPLICATIONS
• Assignee: Jacques Le Roux
• Reporter: Jacques Le Roux
• Created: 05/Feb/10 09:55
• Updated: 25/Apr/10 18:51
• Resolved: 25/Apr/10 18:51

Description

Following Sascha Rodekamp's work on layer lookups OFBIZ-3374 and improvements OFBIZ-3430, I propose now to replace old the popup lookups by layered (Ajaxified) lookups.

For that please find a patch attached. In this patch I followed a simple S/R tactic:

• I replaced all occurences of LookupDecorator by LookupLayerPopupDecorator in screens
• I replaced all occurences of <lookup by <lookup presentation="layer" position="center"

It's as simple as this. For the moment I decided to use as default position="center" because it's was the easiest (sure that any lookups will be out of the screen). I think we will refine this by removing position="center" and use the default (position="normal") which does not move the layer from the point it's called and will be more aesthetic.

I did not test anything in OFBIz OOTB for the moment, but I already use layered lookups in a custom application without any issues so far.

The only drawback I found for the moment is when a lookup is called from a lookup. If you are aware of such cases please chime in.

Of course everybody is encouraged to test this improvement as much as possible. I really think it's a very cool feature for users, and they will appreciate. There are still some ideas like that (see the link Sasca referred to in OFBIZ-3374), and we will try to implement them.

There are issues to be fixed prior to a commit, see OFBIZ-3446.

Issue Links

• is blocked by: OFBIZ-3446 Allow to open a layer lookup from a layer lookup