Openbravo Issue Tracking System - Retail Modules | |||||||||||||||||||
View Issue Details | |||||||||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||||||||
0034648 | Retail Modules | Web POS | public | 2016-11-28 16:40 | 2017-01-08 21:31 | ||||||||||||||
Reporter | dbaz | ||||||||||||||||||
Assigned To | migueldejuana | ||||||||||||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||||||||||||
Status | closed | Resolution | fixed | ||||||||||||||||
Platform | OS | 5 | OS Version | ||||||||||||||||
Product Version | |||||||||||||||||||
Target Version | Fixed in Version | RR17Q1 | |||||||||||||||||
Merge Request Status | |||||||||||||||||||
Review Assigned To | guilleaer | ||||||||||||||||||
OBNetwork customer | |||||||||||||||||||
Support ticket | |||||||||||||||||||
Regression level | |||||||||||||||||||
Regression date | |||||||||||||||||||
Regression introduced in release | |||||||||||||||||||
Regression introduced by commit | |||||||||||||||||||
Triggers an Emergency Pack | No | ||||||||||||||||||
Summary | 0034648: [UX] Ugly line breaks in the 'Receipt' with high prices | ||||||||||||||||||
Description | Right now [Receipt_4-3_Current.png] there are ugly line breaks in the receipt if there are narrow aspect ratios and high prices. There should not be line breaks neither in the prices nor in the quantities. More information in the 'Proposed Solution' section. | ||||||||||||||||||
Steps To Reproduce | Set 393x700 screen resolution and add a line to the receipt with thousands of products. | ||||||||||||||||||
Proposed Solution | The curent implementation has 1 row with 4 columns (product name, quantity, unit price and gross price) and the width of each one is based on a percentage of the receipt width. This solution works ok even with narrow aspects ratios with no-inflated currencies and moderate amounts, but there are problems if the prices are too high. One possible solution could be adding a preference to let two possible visualizations: the current one (that works ok for most of the users around the world) and a new one splitting the receipt line in two rows (for working with inflated currencies), one for the product name and another one for the quantity, unit and price. The problem with this solution is that, in the second case, with wide aspect ratios, there will be two rows when all the quantities probably fit also in one single row. Another better solution is to reach a trade-off between all the variables and have a single visualization that works for all cases. That is the one is going to be developed: As mentioned before, right now there is one single row with 4 columns (40% product name, 20% quantity, 20% unit price, 20% gross price). The idea is to introduce some changes to make these width adapted in different aspect ratios. After performing a researh with several amounts and aspect rations, we have concluded that the rules should be: Landscape aspect ratios: * Aspect ratio < 5:4 - Product name: remaining available width - Quantity: 55px - Unit price: 80px - Gross price: 80px * Aspect ratio > 5:4 and < 3:2 - Product name: remaining available width - Quantity: 55px - Unit price: grows proportionally from 80px to 105px - Gross price: grows proportionally from 80px to 105px * Aspect ratio > 3:2 - Product name: remaining available width - Quantity: 55px - Unit price: 105px - Gross price: 105px Portrait aspect ratios: * Aspect ratio < 10:16 - Product name: remaining available width - Quantity: 55px - Unit price: 80px - Gross price: 80px * Aspect ratio > 10:16 and < 3:4 - Product name: remaining available width - Quantity: 55px - Unit price: grows proportionally from 80px to 105px - Gross price: grows proportionally from 80px to 105px * Aspect ratio > 3:4 - Product name: remaining available width - Quantity: 55px - Unit price: 105px - Gross price: 105px With this configuration, even with 393x700 resolution, there is enough room for 6 digits in 'Quantity' and 7 digits in 'Unit price' and 'Gross price'. If the amount doesn't fit its space, it should shrink its font-size (example: https://jsfiddle.net/0c84u0rg/1/ [^]) and also re-adapt if the browser is resized or if the quantity/price changes. The 'Product Characteristics', 'TOTAL' and 'Taxes' bottom lines should be adapted to these rules too. In the case of 'Product Characteristics' there should be also more changes to ensure a proper visualization: * padding-top: 4px; * font-size: 14px; There is a patch attached [receiptLargeAmounts.diff] that solves several things: * Aspect ratio based rules for the widths in both landscape and portrait mode for product lines, product characteristics, total and taxes. * UI changes of receiptLargeAmounts.diff This patch doesn't solve: * Required changes to shrink the font-size if the quantity/price does not fit the available space. * Changes in 'OB.UI.TotalMultiReceiptLine' * Changes in 'OB.UI.TotalTaxLine' * Changes in 'OB.UI.RenderPaymentLine' There are also attached several mockups showing how the receipt should look like in different aspect ratios [Receipt_4-3.png] [Receipt_16-10.png] [Receipt_16-9.png] | ||||||||||||||||||
Additional Information | |||||||||||||||||||
Tags | No tags attached. | ||||||||||||||||||
Relationships |
| ||||||||||||||||||
Attached Files | Receipt_4-3_Current.png (126,449) 2016-11-28 16:40 https://issues.openbravo.com/file_download.php?file_id=10120&type=bug receiptLargeAmounts.diff (6,310) 2016-11-28 16:41 https://issues.openbravo.com/file_download.php?file_id=10121&type=bug Receipt_Smartphone.png (59,003) 2016-11-28 16:41 https://issues.openbravo.com/file_download.php?file_id=10122&type=bug Receipt_4-3.png (211,431) 2016-11-28 16:41 https://issues.openbravo.com/file_download.php?file_id=10123&type=bug Receipt_16-10.png (228,376) 2016-11-28 16:41 https://issues.openbravo.com/file_download.php?file_id=10124&type=bug Receipt_16-9.png (242,903) 2016-11-28 16:42 https://issues.openbravo.com/file_download.php?file_id=10125&type=bug issue34648Posterminal.commit (8,099) 2016-12-02 13:52 https://issues.openbravo.com/file_download.php?file_id=10150&type=bug issue34648Core.commit (4,658) 2016-12-02 13:52 https://issues.openbravo.com/file_download.php?file_id=10151&type=bug issues.png (114,655) 2016-12-07 21:49 https://issues.openbravo.com/file_download.php?file_id=10164&type=bug org.openbravo.mobile.core_adaptAllFontSizes.diff (3,649) 2016-12-21 21:01 https://issues.openbravo.com/file_download.php?file_id=10207&type=bug | ||||||||||||||||||
Issue History | |||||||||||||||||||
Date Modified | Username | Field | Change | ||||||||||||||||
2016-11-28 16:40 | dbaz | New Issue | |||||||||||||||||
2016-11-28 16:40 | dbaz | Assigned To | => Retail | ||||||||||||||||
2016-11-28 16:40 | dbaz | File Added: Receipt_4-3_Current.png | |||||||||||||||||
2016-11-28 16:40 | dbaz | Triggers an Emergency Pack | => No | ||||||||||||||||
2016-11-28 16:41 | dbaz | File Added: receiptLargeAmounts.diff | |||||||||||||||||
2016-11-28 16:41 | dbaz | File Added: Receipt_Smartphone.png | |||||||||||||||||
2016-11-28 16:41 | dbaz | File Added: Receipt_4-3.png | |||||||||||||||||
2016-11-28 16:41 | dbaz | File Added: Receipt_16-10.png | |||||||||||||||||
2016-11-28 16:42 | dbaz | File Added: Receipt_16-9.png | |||||||||||||||||
2016-11-28 18:07 | dbaz | Proposed Solution updated | |||||||||||||||||
2016-11-28 18:08 | dbaz | Relationship added | blocks 0034538 | ||||||||||||||||
2016-11-28 18:13 | dbaz | Proposed Solution updated | |||||||||||||||||
2016-12-01 00:03 | mario_castello | Assigned To | Retail => mario_castello | ||||||||||||||||
2016-12-01 00:03 | mario_castello | Status | new => acknowledged | ||||||||||||||||
2016-12-01 10:49 | dbaz | Proposed Solution updated | |||||||||||||||||
2016-12-01 15:43 | mario_castello | Status | acknowledged => scheduled | ||||||||||||||||
2016-12-01 17:23 | marvintm | Assigned To | mario_castello => migueldejuana | ||||||||||||||||
2016-12-02 13:52 | migueldejuana | File Added: issue34648Posterminal.commit | |||||||||||||||||
2016-12-02 13:52 | migueldejuana | File Added: issue34648Core.commit | |||||||||||||||||
2016-12-05 22:28 | hgbot | Checkin | |||||||||||||||||
2016-12-05 22:28 | hgbot | Note Added: 0092098 | |||||||||||||||||
2016-12-05 22:28 | hgbot | Status | scheduled => resolved | ||||||||||||||||
2016-12-05 22:28 | hgbot | Resolution | open => fixed | ||||||||||||||||
2016-12-05 22:28 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/b3cff0650a3f3455045ee37ff593646ff1261322 [^] | ||||||||||||||||
2016-12-05 22:31 | hgbot | Checkin | |||||||||||||||||
2016-12-05 22:31 | hgbot | Note Added: 0092101 | |||||||||||||||||
2016-12-05 22:31 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/b3cff0650a3f3455045ee37ff593646ff1261322 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/80b0433fad39092f61b92d35efd6ca22d714dd00 [^] | ||||||||||||||||
2016-12-07 13:40 | guilleaer | Note Added: 0092186 | |||||||||||||||||
2016-12-07 13:40 | guilleaer | Status | resolved => new | ||||||||||||||||
2016-12-07 13:40 | guilleaer | Resolution | fixed => open | ||||||||||||||||
2016-12-07 21:49 | dbaz | File Added: issues.png | |||||||||||||||||
2016-12-07 21:52 | dbaz | Note Added: 0092203 | |||||||||||||||||
2016-12-14 11:06 | hgbot | Checkin | |||||||||||||||||
2016-12-14 11:06 | hgbot | Note Added: 0092336 | |||||||||||||||||
2016-12-14 11:06 | hgbot | Status | new => resolved | ||||||||||||||||
2016-12-14 11:06 | hgbot | Resolution | open => fixed | ||||||||||||||||
2016-12-14 11:06 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/80b0433fad39092f61b92d35efd6ca22d714dd00 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/185d7e14347a4fe8364e8210dae48d6f0409c156 [^] | ||||||||||||||||
2016-12-14 11:08 | hgbot | Checkin | |||||||||||||||||
2016-12-14 11:08 | hgbot | Note Added: 0092337 | |||||||||||||||||
2016-12-14 11:08 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/185d7e14347a4fe8364e8210dae48d6f0409c156 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/55c68c1cfb390b3c04a5b9227fadaa528549602b [^] | ||||||||||||||||
2016-12-20 12:59 | hgbot | Checkin | |||||||||||||||||
2016-12-20 12:59 | hgbot | Note Added: 0092852 | |||||||||||||||||
2016-12-20 12:59 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/55c68c1cfb390b3c04a5b9227fadaa528549602b [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/27386061217c051f04d5bf1daf8a35a80f75564f [^] | ||||||||||||||||
2016-12-20 13:00 | hgbot | Checkin | |||||||||||||||||
2016-12-20 13:00 | hgbot | Note Added: 0092853 | |||||||||||||||||
2016-12-20 13:00 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/27386061217c051f04d5bf1daf8a35a80f75564f [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/df6cc4a270df2d57e184198f4c8ada1066ad123b [^] | ||||||||||||||||
2016-12-20 13:06 | migueldejuana | Note Added: 0092854 | |||||||||||||||||
2016-12-20 13:58 | marvintm | Review Assigned To | => marvintm | ||||||||||||||||
2016-12-20 13:58 | marvintm | Status | resolved => closed | ||||||||||||||||
2016-12-20 13:58 | marvintm | Fixed in Version | => RR17Q1 | ||||||||||||||||
2016-12-20 16:42 | dbaz | Note Added: 0092863 | |||||||||||||||||
2016-12-20 16:42 | dbaz | Status | closed => new | ||||||||||||||||
2016-12-20 16:42 | dbaz | Resolution | fixed => open | ||||||||||||||||
2016-12-20 16:42 | dbaz | Fixed in Version | RR17Q1 => | ||||||||||||||||
2016-12-20 17:30 | marvintm | Note Added: 0092866 | |||||||||||||||||
2016-12-20 17:36 | dbaz | Note Edited: 0092863 | bug_revision_view_page.php?bugnote_id=0092863#r14087 | ||||||||||||||||
2016-12-20 17:37 | dbaz | Note Added: 0092867 | |||||||||||||||||
2016-12-21 12:04 | hgbot | Checkin | |||||||||||||||||
2016-12-21 12:04 | hgbot | Note Added: 0092902 | |||||||||||||||||
2016-12-21 12:04 | hgbot | Status | new => resolved | ||||||||||||||||
2016-12-21 12:04 | hgbot | Resolution | open => fixed | ||||||||||||||||
2016-12-21 12:04 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/df6cc4a270df2d57e184198f4c8ada1066ad123b [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/515b485ea12822cc3a66a91f39b5041ef768bfba [^] | ||||||||||||||||
2016-12-21 12:05 | hgbot | Checkin | |||||||||||||||||
2016-12-21 12:05 | hgbot | Note Added: 0092903 | |||||||||||||||||
2016-12-21 12:05 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/515b485ea12822cc3a66a91f39b5041ef768bfba [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/1dff56facf9b8bdae047cc280d1438bb53b0b566 [^] | ||||||||||||||||
2016-12-21 21:00 | dbaz | Note Added: 0092929 | |||||||||||||||||
2016-12-21 21:00 | dbaz | Status | resolved => new | ||||||||||||||||
2016-12-21 21:00 | dbaz | Resolution | fixed => open | ||||||||||||||||
2016-12-21 21:01 | dbaz | File Added: org.openbravo.mobile.core_adaptAllFontSizes.diff | |||||||||||||||||
2016-12-23 13:32 | hgbot | Checkin | |||||||||||||||||
2016-12-23 13:32 | hgbot | Note Added: 0092972 | |||||||||||||||||
2016-12-23 13:32 | hgbot | Status | new => resolved | ||||||||||||||||
2016-12-23 13:32 | hgbot | Resolution | open => fixed | ||||||||||||||||
2016-12-23 13:32 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/1dff56facf9b8bdae047cc280d1438bb53b0b566 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/22930b9524b7f35364c00a26b4efe8365baa12fb [^] | ||||||||||||||||
2016-12-23 15:09 | marvintm | Review Assigned To | marvintm => guilleaer | ||||||||||||||||
2016-12-23 16:10 | guilleaer | Status | resolved => closed | ||||||||||||||||
2016-12-23 16:10 | guilleaer | Fixed in Version | => RR17Q1 | ||||||||||||||||
2016-12-29 13:47 | hgbot | Checkin | |||||||||||||||||
2016-12-29 13:47 | hgbot | Note Added: 0093053 | |||||||||||||||||
2017-01-08 21:31 | hgbot | Checkin | |||||||||||||||||
2017-01-08 21:31 | hgbot | Note Added: 0093249 | |||||||||||||||||
2018-09-21 12:10 | dbaz | Relationship added | related to 0039338 |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|