[COMPASS-5151] z-index on APB toolbar is too large compared to modal Created: 12/Oct/21  Updated: 29/Oct/23  Resolved: 13/Oct/21

Status: Closed
Project: Compass
Component/s: Aggregation pipeline
Affects Version/s: None
Fix Version/s: No version

Type: Bug Priority: Major - P3
Reporter: Julia Ruddy (Inactive) Assignee: Rhys Howell
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File Screen Shot 2021-10-12 at 10.34.23 AM.png     PNG File Screen Shot 2021-10-12 at 10.34.29 AM.png     PNG File Screen Shot 2021-10-12 at 11.24.11 AM.png    
Issue Links:
Related
Documentation Changes: Not Needed
Sprint: Iteration Novokuybyshevsk

 Description   

Problem Statement/Rationale

When using version 8.13.0 of @mongodb-js/compass-aggregations in MMS, I noticed that the z-index of the toolbar is too large compared to the z-index of the modals. This results in the buttons showing up on top of the modal. Does not appear that this happens inside the compass app, but when using the component on its own this is an issue.

Edit: This also applies when the settings side bar is open and you click the modal.

Steps to Reproduce

Use the AggregationsPlugin and click the new pipeline or "new pipeline from text" buttons. See attached screenshot for example.

Expected Results

Buttons should appear behind the modal.

Actual Results

Buttons are appearing in front of the modal.

Additional Notes

Either the modal needs to explicitly set a z-index higher than those of the buttons or the buttons should not set a z-index at all.



 Comments   
Comment by Githook User [ 18/Oct/21 ]

Author:

{'name': 'Rhys', 'email': 'Anemy@users.noreply.github.com', 'username': 'Anemy'}

Message: fix(compass-aggregations): add z-index to top level aggregations to avoid overlaying the modal COMPASS-5151 (#2508)
Branch: compass-5113-remove-loading-screen-from-the-app
https://github.com/mongodb-js/compass/commit/492c5d49ef10d34906f4eb5f52f2b024cf696270

Comment by Githook User [ 14/Oct/21 ]

Author:

{'name': 'Rhys', 'email': 'Anemy@users.noreply.github.com', 'username': 'Anemy'}

Message: fix(compass-aggregations): add z-index to top level aggregations to avoid overlaying the modal COMPASS-5151 (#2508)
Branch: compass-home-ts
https://github.com/mongodb-js/compass/commit/492c5d49ef10d34906f4eb5f52f2b024cf696270

Comment by Githook User [ 14/Oct/21 ]

Author:

{'name': 'Rhys', 'email': 'Anemy@users.noreply.github.com', 'username': 'Anemy'}

Message: fix(compass-aggregations): add z-index to top level aggregations to avoid overlaying the modal COMPASS-5151 (#2508)
Branch: COMPASS-5053-loading-status
https://github.com/mongodb-js/compass/commit/492c5d49ef10d34906f4eb5f52f2b024cf696270

Comment by Rhys Howell [ 13/Oct/21 ]

Let's do a release of the `compass-aggregations` package to make this fix available to Cloud.

Comment by Githook User [ 13/Oct/21 ]

Author:

{'name': 'Rhys', 'email': 'Anemy@users.noreply.github.com', 'username': 'Anemy'}

Message: fix(compass-aggregations): add z-index to top level aggregations to avoid overlaying the modal COMPASS-5151 (#2508)
Branch: main
https://github.com/mongodb-js/compass/commit/492c5d49ef10d34906f4eb5f52f2b024cf696270

Generated at Wed Feb 07 22:38:37 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.