Skip to content
This repository has been archived by the owner on Dec 13, 2020. It is now read-only.

Larger photo preview screen/ modal overlay #695

Closed
metas-mk opened this issue Apr 27, 2017 · 11 comments
Closed

Larger photo preview screen/ modal overlay #695

metas-mk opened this issue Apr 27, 2017 · 11 comments

Comments

@metas-mk
Copy link
Member

Type of issue

Feature request

Current behavior

Currently, when capturing a photo via the material receipt window, a small preview screen is shown in the bottom right corner.

Expected behavior

The user needs to precisely see if everything is captured on the photo (quality reasons). Please enlarge the preview screen to screen wide overlay.

image

@metas-mk metas-mk added this to the 2017-18 milestone Apr 27, 2017
@metas-mk
Copy link
Member Author

metas-mk commented Apr 27, 2017

Why not directly open the webcam preview (as wide as modal overlay screen) when opening this already existing overlay. the user can then press on image or start button directly to make the foto.

@damianprzygodzki damianprzygodzki self-assigned this Apr 27, 2017
damianprzygodzki added a commit that referenced this issue Apr 27, 2017
metas-ts added a commit to metasfresh/metasfresh that referenced this issue Apr 28, 2017
[#332](metasfresh/metasfresh-webui-api-legacy#332) Empties window without pre-selection possible
[#695](metasfresh/metasfresh-webui-frontend-legacy#695) Lager photo preview screen/ modal overlay
[#328](metasfresh/metasfresh-webui-api-legacy#328) Manufacturing receipt mandatory Field not filled
[#1388](#1388) HU from Cost Collector is locked and cannot be selected anymore after the CC was reversed
[#683](metasfresh/metasfresh-webui-frontend-legacy#683) Tooltip on first level breadcrumb
[#682](metasfresh/metasfresh-webui-frontend-legacy#682) Sidelist opening shall automatically have the first line selected
[#322](metasfresh/metasfresh-webui-api-legacy#322) Manufacturing Issue/Receipt: Fix Qty and QtyPlan columns
[#676](metasfresh/metasfresh-webui-frontend-legacy#676) Process result: implement openDocument as modal
[#1358](#1358) Drop de.metas.order.callout.CheckDouble
[#411](metasfresh/metasfresh-webui-frontend-legacy#411) Make the parameter names visible in filters
@metas-dh
Copy link
Member

metas-dh commented May 2, 2017

Results of IT1
tested on w101

  • webcam preview (as wide as modal overlay screen): smaller than modal overlay screen, see pic, not sure this is big enough?
  • user can then press on image or start button directly to make the foto: pressing on the image worked, but pressing the start button doesn't do anything: NOK
  • allow escape to get back or press outside: not working, also pressing Cancel does not have any affect, i had to mark the URL and press Enter to get back to Wareneingangsdispo: NOK

@metas-mk: is this webcam preview big enough?
2017-05-02 10_28_29-metasfresh

@metas-dh
Copy link
Member

metas-dh commented May 3, 2017

checked with mk: webcam preview is big enough: OK

pls fix the other 2 NOKs (see above)

@metas-mk metas-mk changed the title Lager photo preview screen/ modal overlay Larger photo preview screen/ modal overlay May 3, 2017
@teosarca
Copy link
Member

teosarca commented May 4, 2017

also i think it would look better if you would gray out everything which is behind. wdyt?

@damianprzygodzki
Copy link
Contributor

@metas-dh

user can then press on image or start button directly to make the foto: pressing on the image worked, but pressing the start button doesn't do anything: NOK

The start button shouldn't take a capture. At least i can't see that it is a case of that issue and has been never discussed.

@metas-dh
Copy link
Member

metas-dh commented May 8, 2017

@damianprzygodzki: checked again with mk: nothing happening for pressing Start button is OK

BUT: performance is not acceptable, taking picture when clicking on it takes far too long, used to be much faster!

@damianprzygodzki
Copy link
Contributor

@metas-dh It is not a performance question. We are waiting for the API calls, to close the camera view when everything will be ready.

I can remove that if you want. But then, we have to still wait for the API to load preview.

damianprzygodzki added a commit that referenced this issue May 8, 2017
…ew before api response for image widget
@teosarca
Copy link
Member

teosarca commented May 9, 2017

Hi guys,

took a quick look on this task...
image

  1. frontend it's uploading the webcam photo (~0.5Mb) => backend it's very slow (851ms) => IMHO that's a thing we need to address on backend side
  2. frontend it's querying the image we just uploaded it (took 637ms) => wondering if instead of querying it again, what if frontend would just update directly the image widget:
  • set the image ID received on 1.
  • set the webcam captured image

wdyt?

@damianprzygodzki
Copy link
Contributor

damianprzygodzki commented May 9, 2017

We can, but:

  • it is taking less time than upload (so it is minor issue)
  • and what if there will be some image processing (size/format unification, validation) on the backend side, and we would be automatically update the view?
  • or the error (500, 400 or timeout)

Fetching state from API right after user action make us sure that we are up to date with API state.

@teosarca
Copy link
Member

teosarca commented May 9, 2017

TODO: create another issue to handle the speed improvements.
On this task we can accept the work as it is.

@teosarca
Copy link
Member

teosarca commented May 9, 2017

=> follow-up: metasfresh/metasfresh-webui-api-legacy#350

@teosarca teosarca closed this as completed May 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants