0047914: It is not possible to show a popover with an open popup - Openbravo Issue Tracking System

Openbravo Issue Tracking System - POS2

View Issue Details

ID

Project

Category

View Status

Date Submitted

Last Update

0047914

POS2

Core

public

2021-10-22 09:42

2021-10-28 16:09

Reporter

javierRodriguez

Assigned To

caristu

Priority

normal

Severity

major

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS

5

OS Version

Product Version

Target Version

Fixed in Version

Merge Request Status

Review Assigned To

OBNetwork customer

Support ticket

Regression level

Production - QA Approved

Regression date

Regression introduced in release

Regression introduced by commit

https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/39f0e631b74fc62ee452b1a081bfc3ae14964767 [^]

Triggers an Emergency Pack

No

Summary

0047914: It is not possible to show a popover with an open popup

Description

In this moment, If there is a popup component open and inside it there is an open popover, this one is going to be shown behind the popup.

In the same way, BaseMenu has the same issue.

Steps To Reproduce

1) Open an already paid ticket
2) Navigate to Proofs of Payment
3) Create New > Fill In Amount
4) It is not possible to enter a quantity because the quantity popover is displayed behind the popup

Proposed Solution

Additional Information

Tags

No tags attached.

Relationships

depends on	backport	0047927	TAP	closed	caristu	It is not possible to show a popover with an open popup
causes	defect	0047949		closed	caristu	Quotation can not be rejected because Reject Reasons are not selectable

Attached Files

Issue History

Date Modified

Username

Field

Change

2021-10-22 09:42

javierRodriguez

New Issue

2021-10-22 09:42

javierRodriguez

Assigned To

=> javierRodriguez

2021-10-22 09:42

javierRodriguez

Triggers an Emergency Pack

=> No

2021-10-22 10:09

javierRodriguez

Description Updated

bug_revision_view_page.php?rev_id=23228#r23228

2021-10-22 13:13

javierRodriguez

Steps to Reproduce Updated

bug_revision_view_page.php?rev_id=23234#r23234

2021-10-22 13:13

javierRodriguez

Proposed Solution updated

2021-10-22 13:28

javierRodriguez

Assigned To

javierRodriguez => platform

2021-10-25 11:21

Status

new => acknowledged

2021-10-25 13:16

javierRodriguez

Steps to Reproduce Updated

bug_revision_view_page.php?rev_id=23235#r23235

2021-10-25 16:11

Regression level

=> Production - QA Approved

2021-10-25 16:11

Regression introduced by commit

=> https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/39f0e631b74fc62ee452b1a081bfc3ae14964767 [^]

2021-10-25 16:14

Status

acknowledged => scheduled

2021-10-25 16:14

Status

scheduled => acknowledged

2021-10-26 16:20

Note Added: 0132652

2021-10-26 16:20

Assigned To

platform => caristu

2021-10-27 13:24

Note Added: 0132684

2021-10-27 14:22

Steps to Reproduce Updated

bug_revision_view_page.php?rev_id=23242#r23242

2021-10-27 15:37

Status

acknowledged => scheduled

2021-10-28 12:18

Note Added: 0132705

2021-10-28 13:43

Resolution

open => fixed

2021-10-28 13:43

Status

scheduled => closed

2021-10-28 13:43

Note Added: 0132708

2021-10-28 13:43

Note Added: 0132709

2021-10-28 13:43

Note Added: 0132710

2021-10-28 13:43

Note Added: 0132711

2021-10-28 13:43

Note Added: 0132712

2021-10-28 13:43

Note Added: 0132713

2021-10-28 13:43

Note Added: 0132714

2021-10-28 16:09

Note Added: 0132717

2021-11-02 08:11

Relationship added

causes 0047949

Notes

Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2 [^]
Changeset: 3a4c6bf99fef5bb1848869330569cee1fc6e6664
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 2021-10-28T08:36:55+02:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/commit/3a4c6bf99fef5bb1848869330569cee1fc6e6664 [^]

fixes ISSUE-47914: Auto order popups regarding the UserActionScrim

Now the UserActionScrim component has z-index greater than all the
BaseDialog and BasePopoper components. This way by default they are all
opened behind the scrim so we can lock them if they are already opened
when launching a user action.

Together with this, if one of those components is opened while a user
action is being executed, in this case they are opened with a z-index
higher than the UserActionScrim. This way we prevent locking dialogs
which are opened as part of the user action execution, for example for
requesting a confirmation or data coming from the user.

---
A web-jspack/org.openbravo.core2/src/hooks/useLockableStackOrder/__test__/useLockableStackOrder.test.jsx
A web-jspack/org.openbravo.core2/src/hooks/useLockableStackOrder/index.js
A web-jspack/org.openbravo.core2/src/hooks/useLockableStackOrder/useLockableStackOrder.js
M web-jspack/org.openbravo.core2/src/components/BaseDialog/BaseDialog.jsx
M web-jspack/org.openbravo.core2/src/components/BaseDialog/BaseDialog.scss
M web-jspack/org.openbravo.core2/src/components/BaseDialog/__test__/BaseDialog.test.jsx
M web-jspack/org.openbravo.core2/src/components/BasePopover/BasePopover.jsx
M web-jspack/org.openbravo.core2/src/components/BasePopover/BasePopover.scss
M web-jspack/org.openbravo.core2/src/components/BasePopover/__test__/BasePopover.test.jsx
M web-jspack/org.openbravo.core2/src/components/BasePopover/stories/BasePopover.stories.jsx
M web-jspack/org.openbravo.core2/src/components/BasePopover/stories/MockBasePopover.jsx
M web-jspack/org.openbravo.core2/src/components/Scrim/UserActionScrim/UserActionScrim.scss
---