Openbravo Issue Tracking System - POS2
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0050942POS2POSpublic2022-11-17 17:402022-11-22 16:33
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0050942: Layout of VideoInputHidden in MainPOSAppConfig should be updated
DescriptionThis fix [1] is going to change the layout of the MainPOSAppConfig, among other things. That layout is going to change from having 12 columns to having only 3.

The problem is that the problem gets in conflict with this other change [2], where a hidden component is added to the 12th row. (see attached image how the layout gets broken)

[1] https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1374/diffs#c4c2f31edfd300019e5ca222b625f93a31a880be_7_7 [^]
[2] https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.camerabarcodescanner.posterminal/-/commit/9736a67a9382cfb8065d8fd3e702745fdddf928d#43c1dd8d37dd48ab336898c37aa9adf942731c93_52_65 [^]
Steps To Reproduce- In a POS2 module that contains the org.openbravo.retail.camerabarcodescanner.posterminal module, install the branches of these MRs: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/merge_requests/917, [^] https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1374 [^]
- Open POS2 and see that the layout is broken, like in the attached image
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Filespng BrokenLayout.png (152,018) 2022-11-17 17:43
https://issues.openbravo.com/file_download.php?file_id=17848&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2022-11-17 17:40AugustoMauchNew Issue
2022-11-17 17:40AugustoMauchAssigned To => Retail
2022-11-17 17:40AugustoMauchTriggers an Emergency Pack => No
2022-11-17 17:40AugustoMauchStatusnew => scheduled
2022-11-17 17:43AugustoMauchFile Added: BrokenLayout.png
2022-11-17 17:48hgbotNote Added: 0143801
2022-11-17 17:54AugustoMauchAssigned ToRetail => AugustoMauch
2022-11-22 16:33hgbotResolutionopen => fixed
2022-11-22 16:33hgbotStatusscheduled => closed
2022-11-22 16:33hgbotNote Added: 0144013
2022-11-22 16:33hgbotNote Added: 0144014

Notes
(0143801)
hgbot   
2022-11-17 17:48   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.camerabarcodescanner.posterminal/-/merge_requests/5 [^]
(0144013)
hgbot   
2022-11-22 16:33   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.camerabarcodescanner.posterminal [^]
Changeset: aac33310139ea83ca725f97c5aaf8fa9bb2f6a19
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 17-11-2022 17:47:40
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.camerabarcodescanner.posterminal/-/commit/aac33310139ea83ca725f97c5aaf8fa9bb2f6a19 [^]

Fixes ISSUE-50942: Updates position in layout to prevent broken layout

The number of rows of the Main POS Layout is going to be reduced to 3, so the OBCBS_VideoInputHidden being in
row 12 was breaking the layout. It has been moved to row 1, even if it overlaps with other components it does
not matter because the component will be hidden

---
M web-jspack/org.openbravo.retail.camerabarcodescanner.posterminal/src/ob-init.js
---
(0144014)
hgbot   
2022-11-22 16:33   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.camerabarcodescanner.posterminal/-/merge_requests/5 [^]