Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 10.3
    • Component/s: Web UI

      Description

      Problem
      Drop zone to replace main file takes to much space. It could be used to show a bigger preview.

      Solution
      Look into mockups attached or look into https://projects.invisionapp.com/freehand/document/3D8jO6dgO

      In terms of main file replacement, I would still consider 2 possible behaviors:
      A) Action (icon attached) that will reveal a pop up with extended options to upload new file. The icon, if possible, should be ordered right before the remove blob icon.
      B) Have a droparea on top of the image-viewer, triggered when a file is dragged on top of it.

      Note: the 'file' label on top of the dialog, would be a variable that would always assume the name of the file that is being replaced, to provide more certainity to the user of what he is actually replacing.

        Attachments

        1. ic-replace.svg
          2 kB
        2. New_Replace_A-overlay.png
          New_Replace_A-overlay.png
          763 kB
        3. New_Replace1.png
          New_Replace1.png
          764 kB
        4. New_Replace2-hover.png
          New_Replace2-hover.png
          766 kB
        5. New_Replace3.png
          New_Replace3.png
          308 kB
        6. New_Replace4 Dialog.png
          New_Replace4 Dialog.png
          299 kB

          Issue Links

            Activity

            Hide
            gbarata Gabriel Barata added a comment -

            Option A forces an extra step to replace the blob. Option B seems to be the most usable, and it's what we had two years ago. But it has two major problems:
            1) The overlay won't work well with iframes; it will require negative positioning and overflowing width and height to create a margin where the drag event can be picked up.
            2) It needs a confirmation popup on drop, to prevent accidental replacement of the blob.

            Show
            gbarata Gabriel Barata added a comment - Option A forces an extra step to replace the blob. Option B seems to be the most usable, and it's what we had two years ago. But it has two major problems: 1) The overlay won't work well with iframes; it will require negative positioning and overflowing width and height to create a margin where the drag event can be picked up. 2) It needs a confirmation popup on drop, to prevent accidental replacement of the blob.
            Hide
            grenard Guillaume Renard added a comment - - edited

            After discussing with Tiago Cardoso it's been agreed to do option A

            A) Action (icon attached) that will reveal a pop up with extended options to upload new file. The icon, if possible, should be ordered right before the remove blob icon.

            Are we really sure we want to backport such a change of behavior?

            Show
            grenard Guillaume Renard added a comment - - edited After discussing with Tiago Cardoso it's been agreed to do option A A) Action (icon attached) that will reveal a pop up with extended options to upload new file. The icon, if possible, should be ordered right before the remove blob icon. Are we really sure we want to backport such a change of behavior?
            Hide
            grenard Guillaume Renard added a comment - - edited

            Adding the grooming tag back since the description is not clear at all for what should be done in a 9.10 backport as the behavior may not be the same than on master !?

            Show
            grenard Guillaume Renard added a comment - - edited Adding the grooming tag back since the description is not clear at all for what should be done in a 9.10 backport as the behavior may not be the same than on master !?
            Hide
            grenard Guillaume Renard added a comment - - edited

            No backport afterall. We'll create the mentioned action element for option A and let people know how to add it for 9.10 in their project.

            Show
            grenard Guillaume Renard added a comment - - edited No backport afterall. We'll create the mentioned action element for option A and let people know how to add it for 9.10 in their project.
            Hide
            hudson Jenkins added a comment -

            SUCCESS: Integrated in master/addons_nuxeo-web-ui-master #1272
            NXP-25638: add replace button and corresponding action (nmpcunha: 355e815e650c617301e012c5ac5eecdaf8130ca0)

            Show
            hudson Jenkins added a comment - SUCCESS: Integrated in master/addons_nuxeo-web-ui-master #1272 NXP-25638 : add replace button and corresponding action (nmpcunha: 355e815e650c617301e012c5ac5eecdaf8130ca0 )

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 2 days, 1 hour, 30 minutes
                  2d 1h 30m