Details

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

      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.

        Attachments

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

          Issue Links

            Activity

              People

              • Assignee:
                jacques.le.roux Jacques Le Roux
                Reporter:
                jacques.le.roux Jacques Le Roux
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: