Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Notebook] Delete Page PopUp Does Not Go Away #5679

Closed
1 of 7 tasks
khalidadil opened this issue Aug 18, 2022 · 4 comments · Fixed by #5680
Closed
1 of 7 tasks

[Notebook] Delete Page PopUp Does Not Go Away #5679

khalidadil opened this issue Aug 18, 2022 · 4 comments · Fixed by #5680

Comments

@khalidadil
Copy link
Contributor

Summary

This has been copied from VIPEROMCT-160:
When clicked on the Shift Log for 20220729, the Delete Page popup appeared. When I subsequently clicked elsewhere it remained. Performed the following to try to clear it, which all failed:

Making a shift log entry
Switching to different days and different shifts
Navigating away to a different screen/layout
Refreshing the browser did clear it. And could not get the popup to stick.

Note from Khalid:
I was unable to get the delete popup to display while clicking a notebook page, but I was able to reproduce this while clicking the dropdown next to a notebook page twice. In this case the popup persists as described and nothing can be done to close it other than refreshing the page.

Steps to Reproduce

  1. Go to a notebook and create a section and a page if none exist.
  2. Click on the dropdown arrow next to a notebook page. The "Delete Page" dropdown should open.
  3. Click on the dropdown arrow next to the same notebook page again (avoiding clicking on the "Delete Page" dropdown).
  4. The dropdown will now persist despite trying to click elsewhere and there should be an error in the log.

Environment

  • Open MCT Version: release/2.0.8
  • OS: MacOS
  • Browser: Chrome

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

Delete.Page.Popup.Persisting.mov
@unlikelyzero
Copy link
Collaborator

Will need to address #5713 in a follow up

@jvigliotta
Copy link
Contributor

Verified Fixed - Testathon 9/12/22

@michaelrogers
Copy link
Contributor

Fix verified during testathon 09/12/2022.

@rukmini-bose
Copy link
Contributor

Verified Testathon 9/12/2022

@unlikelyzero unlikelyzero added this to the Target:2.1.1 milestone Feb 16, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants