Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034327Retail ModulesWeb POSpublic2016-10-27 22:132017-05-09 16:44
Reporterdbaz 
Assigned Toranjith_qualiantech_com 
PriorityurgentSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q1 
Merge Request Status
Review Assigned Todbaz
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034327: [UX] If popups are not working properly, an error message should be shown
DescriptionIf popups are not working properly, an error message should be shown.

In issue 32975 there is a flow described that once it is followed, the popups stop working. Since it is a known issue, a message warning the user should be shown in order to avoid confuse him.

The message should have some clues about the problem origin, something like: "Due to the previous window size change, popups may not work properly. Reload the application to fix them or continue working at your own risk."

Also it should have both "Reload" and "Continue" buttons that allow the user refresh the application or continue opening the popup even with the known error.

With this error popup, the user can know which triggers the error (opening a popup) and why it has happened (the previous window resize or rotation). The user has also the workaround (the 'Reload' button) one click away and can take preventive measures (avoid window resizes) to prevent it in the future.
Steps To ReproduceThe same described in issue 32975
Proposed SolutionI attach several tentative mockups.

Note that if the user decides to continue after the red popup, a warning should be shown in the SCAN area. If this warning box is pressed, the red popup will be shown again.
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0029811 new Retail Better handling of critical, application breaking errors 
related to design defect 0032975 closed guilleaer [SER QA 969] Search popups in web pos (Receipts/BPs/Locations) stop working if they are opened and the the pos window is resized 
blocks defect 0035961RR17Q3 closed guilleaer Due to previous fix, popups are closed if a resize event occurs while a popup is opened 
causes design defect 0034786 closed guilleaer Payment Method Popup ID Duplicated After Completing Ticket 
Attached Filespng PopupError_v1_01.png (113,133) 2016-10-27 22:14
https://issues.openbravo.com/file_download.php?file_id=9980&type=bug
png

png PopupError_v1_02.png (79,080) 2016-10-27 22:14
https://issues.openbravo.com/file_download.php?file_id=9981&type=bug
png

png PopupError_v1_03.png (117,147) 2016-10-27 22:14
https://issues.openbravo.com/file_download.php?file_id=9982&type=bug
png

png WarningIcon.png (1,618) 2016-10-27 22:14
https://issues.openbravo.com/file_download.php?file_id=9983&type=bug
png

png Bad_1.png (73,779) 2016-12-23 18:09
https://issues.openbravo.com/file_download.php?file_id=10221&type=bug
png

png Bad_2.png (63,393) 2016-12-23 18:09
https://issues.openbravo.com/file_download.php?file_id=10222&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2016-10-27 22:13dbazNew Issue
2016-10-27 22:13dbazAssigned To => Retail
2016-10-27 22:13dbazTriggers an Emergency Pack => No
2016-10-27 22:14dbazFile Added: PopupError_v1_01.png
2016-10-27 22:14dbazFile Added: PopupError_v1_02.png
2016-10-27 22:14dbazFile Added: PopupError_v1_03.png
2016-10-27 22:14dbazFile Added: WarningIcon.png
2016-10-27 22:15dbazProposed Solution updated
2016-10-27 22:15dbazRelationship addedrelated to 0029811
2016-10-27 22:16dbazRelationship addedrelated to 0032975
2016-10-28 08:56marvintmResolution time => 1478818800
2016-11-21 16:09ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2016-11-21 16:10ranjith_qualiantech_comStatusnew => scheduled
2016-11-30 09:56hgbotCheckin
2016-11-30 09:56hgbotNote Added: 0091919
2016-11-30 09:57hgbotCheckin
2016-11-30 09:57hgbotNote Added: 0091920
2016-11-30 09:57hgbotStatusscheduled => resolved
2016-11-30 09:57hgbotResolutionopen => fixed
2016-11-30 09:57hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fd70eb42be7d91c9e271a78061fb64d159015549 [^]
2016-11-30 09:57hgbotCheckin
2016-11-30 09:57hgbotNote Added: 0091921
2016-12-20 18:33guilleaerReview Assigned To => guilleaer
2016-12-20 18:33guilleaerStatusresolved => closed
2016-12-20 18:33guilleaerFixed in Version => RR17Q1
2016-12-23 18:09dbazNote Added: 0092985
2016-12-23 18:09dbazStatusclosed => new
2016-12-23 18:09dbazResolutionfixed => open
2016-12-23 18:09dbazFixed in VersionRR17Q1 =>
2016-12-23 18:09dbazFile Added: Bad_1.png
2016-12-23 18:09dbazFile Added: Bad_2.png
2016-12-26 10:23ranjith_qualiantech_comStatusnew => scheduled
2016-12-28 12:52hgbotCheckin
2016-12-28 12:52hgbotNote Added: 0093039
2016-12-28 12:52hgbotStatusscheduled => resolved
2016-12-28 12:52hgbotResolutionopen => fixed
2016-12-28 12:52hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fd70eb42be7d91c9e271a78061fb64d159015549 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1fa59de90481be22a169877865f6a875bded4116 [^]
2016-12-28 19:08dbazNote Added: 0093044
2016-12-28 19:08dbazStatusresolved => new
2016-12-28 19:08dbazResolutionfixed => open
2016-12-29 06:47ranjith_qualiantech_comStatusnew => scheduled
2016-12-29 10:01hgbotCheckin
2016-12-29 10:01hgbotNote Added: 0093048
2016-12-29 10:01hgbotCheckin
2016-12-29 10:01hgbotNote Added: 0093049
2016-12-29 10:01hgbotStatusscheduled => resolved
2016-12-29 10:01hgbotResolutionopen => fixed
2016-12-29 10:01hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1fa59de90481be22a169877865f6a875bded4116 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/cf984561a11d2b7941edeee49540b7f05d443534 [^]
2016-12-29 10:05guilleaerReview Assigned Toguilleaer => dbaz
2016-12-29 12:35dbazNote Added: 0093050
2016-12-29 12:35dbazStatusresolved => closed
2016-12-29 12:35dbazFixed in Version => RR17Q2
2016-12-30 21:58dbazFixed in VersionRR17Q2 => RR17Q1
2017-01-09 10:02ranjith_qualiantech_comRelationship addedcauses 0034786
2017-05-09 16:43guilleaerRelationship addedblocks 0035961
2017-05-09 16:44guilleaerNote Added: 0096496

Notes
(0091919)
hgbot   
2016-11-30 09:56   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 9406f2e966a2df5adfd70fddc83ec6161aea1b66
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Nov 29 10:31:47 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/9406f2e966a2df5adfd70fddc83ec6161aea1b66 [^]

Related to issue 34327 : Validate popup while resize screen

* User should be warned if popup is not working while resizing the window
* Add Warning Icon

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M web/org.openbravo.mobile.core/source/component/ob-commonbuttons.js
A web/org.openbravo.mobile.core/assets/img/Warning.png
---
(0091920)
hgbot   
2016-11-30 09:57   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: fd70eb42be7d91c9e271a78061fb64d159015549
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Nov 30 14:25:08 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fd70eb42be7d91c9e271a78061fb64d159015549 [^]

Fixes issue 34327 : Modified Scan area to add button for not working popup

* Warning Message should be shown to the user, when the popup is not working after resize

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M web/org.openbravo.retail.posterminal/js/pointofsale/view/scan.js
---
(0091921)
hgbot   
2016-11-30 09:57   
Repository: tools/automation/pi-mobile
Changeset: 9ce321255eb909756086709a7540b1de851cfca1
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Nov 30 14:20:45 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/9ce321255eb909756086709a7540b1de851cfca1 [^]

Verifies issue 34327 : Added automated test 'I34327_VerifyScanAreaNotification'

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/TestIdPack.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I29260_InDevelopmentMessage.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I34327_VerifyScanAreaNotification.java
---
(0092985)
dbaz   
2016-12-23 18:09   
In the popup, the buttons font-color should be the same red than the popup background. Now they are green (Bad_1.png)

The message button in the SCAN area (Bad_2.png):
* Should have the text vertically aligned (now it is more top-aligned)
* The background should be also red
* It shouldn't have borders. The buttons in the WebPOS have no borders, and the hover status should be reflected with an opacity: 0.75
(0093039)
hgbot   
2016-12-28 12:52   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 1fa59de90481be22a169877865f6a875bded4116
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Wed Dec 28 17:21:57 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1fa59de90481be22a169877865f6a875bded4116 [^]

Fixes issue 34327 : Modified Scan area to add button for not working popup

* Replace Button with Small Button
*

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/scan.js
---
(0093044)
dbaz   
2016-12-28 19:08   
This point of my previous comment it is still pending:

"In the popup, the buttons font-color should be the same red than the popup background. Now they are green (Bad_1.png)"
(0093048)
hgbot   
2016-12-29 10:01   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 72419ebe2b9980e146c7601ee7df7a4ae5d0fe8b
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Dec 29 14:30:50 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/72419ebe2b9980e146c7601ee7df7a4ae5d0fe8b [^]

Related to issue 34327 : Added Button attributes in showConfirmation popup

* Added Style attribute to showConfirmation popup buttons

---
M web/org.openbravo.mobile.core/source/utils/ob-utilitiesui.js
---
(0093049)
hgbot   
2016-12-29 10:01   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: cf984561a11d2b7941edeee49540b7f05d443534
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Dec 29 14:31:07 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/cf984561a11d2b7941edeee49540b7f05d443534 [^]

Fixes issue 34327 : Modified not working popup CSS

* Added Red color to Popup Buttons

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/scan.js
---
(0093050)
dbaz   
2016-12-29 12:35   
Reviewed @ changeset: 7759 - cf984561a11d
(0096496)
guilleaer   
2017-05-09 16:44   
This code has been removed because finally we have a fix for popups:
https://issues.openbravo.com/view.php?id=35961 [^]