• Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Trunk
    • Component/s: ALL APPLICATIONS
    • Labels:


      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.

      1. Min-max issue from FTL.jpg
        31 kB
        Jacques Le Roux
      2. LayoutFixMinMaxTomah.patch
        0.4 kB
        Sascha Rodekamp
      3. LayoutFixes.patch
        4 kB
        Sascha Rodekamp
      4. LayoutFixes.patch
        3 kB
        Sascha Rodekamp
      5. OFBIZ-3442 replace popup lookups by layered lookups.patch
        9 kB
        Jacques Le Roux
      6. ASF.LICENSE.NOT.GRANTED--OFBIZ-3442 replace popup lookups by layered lookups.patch
        9 kB
        Jacques Le Roux

        Issue Links



            • Assignee:
              Jacques Le Roux
              Jacques Le Roux
            • Votes:
              0 Vote for this issue
              1 Start watching this issue


              • Created: