Description of problem:
To ensure performance,the notification drawer should not render when closed. Currently it will render, but "display:none". This does a lot of work creating DOM nodes and listeners that may have an impact on the page. Simpler list pages may not be affected, but complex pages such as monitoring that have charts and heavy visuals may experience greater impact.
@bpeterse the polling still needs to occur so as to inform the notification badge with current alert states, but you are right, there is no need to display the contents of the drawer. Adding code to conditionally render the drawer contents.
We've profiled this, and the two main performance problems we've seen are:
1. Using `AsyncComponent`, which slows page load since the component will always be loaded. This causes the page to flicker full black with a loading icon on console load.
2. Unnecessary component renders due to hook dependencies changing every time.
We can evaluate the performance impact using CSS to hide the drawer has, but I think we should fix the clear problems we've seen during profiling first.
Checked the notification works well with the performance improvement.
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.
For information on the advisory, and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.