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

Announcement Menu Open Location Adjustment #27685

Closed
damen-dotcms opened this issue Feb 21, 2024 · 3 comments · Fixed by #27758 or #27793
Closed

Announcement Menu Open Location Adjustment #27685

damen-dotcms opened this issue Feb 21, 2024 · 3 comments · Fixed by #27758 or #27793

Comments

@damen-dotcms
Copy link
Contributor

damen-dotcms commented Feb 21, 2024

Parent Issue

#25733

Problem Statement

The Announcement Menu needs to open at the same place as the Notifications Menu.

The Announcement Icon should be visible at all times, like the Notification Icon is when you open the Notification Menu.

Screen.Recording.2024-02-21.at.12.49.16.PM.mov

Steps to Reproduce

Open the Announcements Menu. I'm using a 4k monitor set to look like a 1080p.

image

Acceptance Criteria

Menu location should match the prototype.

dotCMS Version

Latest QA.

Proposed Objective

User Experience

Proposed Priority

Priority 3 - Average

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

No response

Assumptions & Initiation Needs

No response

Quality Assurance Notes & Workarounds

No response

Sub-Tasks & Estimates

No response

@manuelrojas manuelrojas self-assigned this Feb 27, 2024
@manuelrojas manuelrojas linked a pull request Feb 29, 2024 that will close this issue
3 tasks
@manuelrojas
Copy link
Contributor

Internal QA Notes:

A pull request has been submitted to reintroduce the about links. Please review to ensure that the Knowledge Center and Contact Us links are appearing correctly.

image

manuelrojas added a commit that referenced this issue Mar 1, 2024
@manuelrojas manuelrojas linked a pull request Mar 1, 2024 that will close this issue
3 tasks
github-merge-queue bot pushed a commit that referenced this issue Mar 1, 2024
@nollymar nollymar reopened this Mar 1, 2024
@dcolina dcolina self-assigned this Mar 4, 2024
@dcolina
Copy link
Contributor

dcolina commented Mar 4, 2024

Internal QA: Passed.

As we can see in the following image. The reported issue has been solved.

Screenshot 2024-03-04 at 18 04 38

@dcolina dcolina closed this as completed Mar 4, 2024
@dcolina dcolina removed their assignment Mar 4, 2024
@bryanboza
Copy link
Member

Fixed, tested on master // Docker // FF

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment