Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0034327 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
design defect | [Retail Modules] Web POS | minor | always | 2016-10-27 22:13 | 2017-05-09 16:44 | |||
Reporter | dbaz | View Status | public | |||||
Assigned To | ranjith_qualiantech_com | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | RR17Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | cf984561a11d | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | dbaz | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0034327: [UX] If popups are not working properly, an error message should be shown | |||||||
Description | If 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 Reproduce | The same described in issue 32975 | |||||||
Proposed Solution | I 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. | |||||||
Tags | No tags attached. | |||||||
Attached Files | PopupError_v1_01.png [^] (113,133 bytes) 2016-10-27 22:14
PopupError_v1_02.png [^] (79,080 bytes) 2016-10-27 22:14 PopupError_v1_03.png [^] (117,147 bytes) 2016-10-27 22:14 WarningIcon.png [^] (1,618 bytes) 2016-10-27 22:14 Bad_1.png [^] (73,779 bytes) 2016-12-23 18:09 Bad_2.png [^] (63,393 bytes) 2016-12-23 18:09 | |||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||||||||||||||||||||||
|
Notes | |
(0091919) hgbot (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 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 (developer) 2016-12-29 12:35 |
Reviewed @ changeset: 7759 - cf984561a11d |
(0096496) guilleaer (manager) 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 [^] |
Issue History | |||
Date Modified | Username | Field | Change |
2016-10-27 22:13 | dbaz | New Issue | |
2016-10-27 22:13 | dbaz | Assigned To | => Retail |
2016-10-27 22:13 | dbaz | Triggers an Emergency Pack | => No |
2016-10-27 22:14 | dbaz | File Added: PopupError_v1_01.png | |
2016-10-27 22:14 | dbaz | File Added: PopupError_v1_02.png | |
2016-10-27 22:14 | dbaz | File Added: PopupError_v1_03.png | |
2016-10-27 22:14 | dbaz | File Added: WarningIcon.png | |
2016-10-27 22:15 | dbaz | Proposed Solution updated | |
2016-10-27 22:15 | dbaz | Relationship added | related to 0029811 |
2016-10-27 22:16 | dbaz | Relationship added | related to 0032975 |
2016-10-28 08:56 | marvintm | Resolution time | => 1478818800 |
2016-11-21 16:09 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
2016-11-21 16:10 | ranjith_qualiantech_com | Status | new => scheduled |
2016-11-30 09:56 | hgbot | Checkin | |
2016-11-30 09:56 | hgbot | Note Added: 0091919 | |
2016-11-30 09:57 | hgbot | Checkin | |
2016-11-30 09:57 | hgbot | Note Added: 0091920 | |
2016-11-30 09:57 | hgbot | Status | scheduled => resolved |
2016-11-30 09:57 | hgbot | Resolution | open => fixed |
2016-11-30 09:57 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fd70eb42be7d91c9e271a78061fb64d159015549 [^] |
2016-11-30 09:57 | hgbot | Checkin | |
2016-11-30 09:57 | hgbot | Note Added: 0091921 | |
2016-12-20 18:33 | guilleaer | Review Assigned To | => guilleaer |
2016-12-20 18:33 | guilleaer | Status | resolved => closed |
2016-12-20 18:33 | guilleaer | Fixed in Version | => RR17Q1 |
2016-12-23 18:09 | dbaz | Note Added: 0092985 | |
2016-12-23 18:09 | dbaz | Status | closed => new |
2016-12-23 18:09 | dbaz | Resolution | fixed => open |
2016-12-23 18:09 | dbaz | Fixed in Version | RR17Q1 => |
2016-12-23 18:09 | dbaz | File Added: Bad_1.png | |
2016-12-23 18:09 | dbaz | File Added: Bad_2.png | |
2016-12-26 10:23 | ranjith_qualiantech_com | Status | new => scheduled |
2016-12-28 12:52 | hgbot | Checkin | |
2016-12-28 12:52 | hgbot | Note Added: 0093039 | |
2016-12-28 12:52 | hgbot | Status | scheduled => resolved |
2016-12-28 12:52 | hgbot | Resolution | open => fixed |
2016-12-28 12:52 | hgbot | Fixed in SCM revision | http://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:08 | dbaz | Note Added: 0093044 | |
2016-12-28 19:08 | dbaz | Status | resolved => new |
2016-12-28 19:08 | dbaz | Resolution | fixed => open |
2016-12-29 06:47 | ranjith_qualiantech_com | Status | new => scheduled |
2016-12-29 10:01 | hgbot | Checkin | |
2016-12-29 10:01 | hgbot | Note Added: 0093048 | |
2016-12-29 10:01 | hgbot | Checkin | |
2016-12-29 10:01 | hgbot | Note Added: 0093049 | |
2016-12-29 10:01 | hgbot | Status | scheduled => resolved |
2016-12-29 10:01 | hgbot | Resolution | open => fixed |
2016-12-29 10:01 | hgbot | Fixed in SCM revision | http://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:05 | guilleaer | Review Assigned To | guilleaer => dbaz |
2016-12-29 12:35 | dbaz | Note Added: 0093050 | |
2016-12-29 12:35 | dbaz | Status | resolved => closed |
2016-12-29 12:35 | dbaz | Fixed in Version | => RR17Q2 |
2016-12-30 21:58 | dbaz | Fixed in Version | RR17Q2 => RR17Q1 |
2017-01-09 10:02 | ranjith_qualiantech_com | Relationship added | causes 0034786 |
2017-05-09 16:43 | guilleaer | Relationship added | blocks 0035961 |
2017-05-09 16:44 | guilleaer | Note Added: 0096496 |
Copyright © 2000 - 2009 MantisBT Group |