Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Modal Management Enhancement for Announcements #27472

Closed
bryanboza opened this issue Jan 30, 2024 · 3 comments · Fixed by #27707
Closed

Modal Management Enhancement for Announcements #27472

bryanboza opened this issue Jan 30, 2024 · 3 comments · Fixed by #27707

Comments

@bryanboza
Copy link
Member

bryanboza commented Jan 30, 2024

Parent Issue

#25733

Problem Statement

In dotCMS, there is an issue where the announcements modal does not automatically close when users open another window, leading to overlap between the announcements modal and the new window. This behavior can be confusing and negatively impact the user experience.

Screenshot 2024-01-30 at 1 31 06 PM

Steps to Reproduce

  • Log into the dotCMS backend with the necessary credentials.
  • Open the announcements modal.
  • Without closing the announcements modal, open another window or perform an action that triggers a new window.
  • Observe that the announcements modal overlaps with the new window, causing a visual conflict.

Acceptance Criteria

  • Automatic Modal Closure: Implement functionality that automatically closes the announcements modal when users open another window, preventing overlap and ensuring a clean user interface.
  • User-Friendly Transition: Ensure that the modal closure occurs smoothly, providing users with a seamless transition between windows.

dotCMS Version

Tested on master // Docker // FF

Proposed Objective

Quality Assurance

Proposed Priority

Priority 2 - Important

@damen-dotcms
Copy link
Contributor

Yeah, as soon as you click anywhere outside of the menu, it should close the menu.

This way, if one menu is open, and you click on another, the original menu would close.

@nollymar nollymar moved this from Next 1-3 Sprints to Current Sprint Backlog in dotCMS - Product Planning Feb 13, 2024
@nollymar nollymar removed the Triage label Feb 13, 2024
@nollymar nollymar moved this from Current Sprint Backlog to In Progress in dotCMS - Product Planning Feb 21, 2024
@manuelrojas manuelrojas moved this from In Progress to In Review in dotCMS - Product Planning Feb 23, 2024
@manuelrojas manuelrojas linked a pull request Feb 23, 2024 that will close this issue
3 tasks
@github-project-automation github-project-automation bot moved this from In Review to Done in dotCMS - Product Planning Feb 23, 2024
@nollymar nollymar reopened this Feb 26, 2024
@github-project-automation github-project-automation bot moved this from Done to Current Sprint Backlog in dotCMS - Product Planning Feb 26, 2024
@nollymar nollymar moved this from Current Sprint Backlog to Internal QA in dotCMS - Product Planning Feb 26, 2024
@dcolina dcolina self-assigned this Feb 26, 2024
@dcolina
Copy link
Contributor

dcolina commented Feb 26, 2024

Internal QA: PASSED.

As we can see in the following video, the functionally works as expected it.

Screen.Recording.2024-02-26.at.18.35.21.mov

@dcolina dcolina removed their assignment Feb 26, 2024
@dcolina dcolina moved this from Internal QA to QA - Backlog in dotCMS - Product Planning Feb 26, 2024
@github-project-automation github-project-automation bot moved this from QA - Backlog to Done in dotCMS - Product Planning Feb 27, 2024
@nollymar nollymar moved this from Done to QA - Backlog in dotCMS - Product Planning Feb 27, 2024
@bryanboza
Copy link
Member Author

Fixed, now we are closing this modal when you click outside. Tested on mater // Docker // FF

@bryanboza bryanboza moved this from QA - Backlog to Done in dotCMS - Product Planning Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants