Bug #20311
closedNotification drawer fuctionality does not seem to be appropriate.
Description
1) If the notification drawer is open and you forget to close it, it remains open even if you browse to some other page.
2) Notification drawer has a weird placement, as in its far from its icon(i am not sure why that is happening).
3) Notification drawer does not follow the pattern fly standards.
Files
Updated by Ohad Levy over 6 years ago
Rahul Bajaj wrote:
1) If the notification drawer is open and you forget to close it, it remains open even if you browse to some other page.
That was by design, you could argue against it.
2) Notification drawer has a weird placement, as in its far from its icon(i am not sure why that is happening).
Afair this happens in patternfly too.
3) Notification drawer does not follow the pattern fly standards.
What exactly?
Updated by Rahul Bajaj over 6 years ago
Ohad Levy wrote:
Rahul Bajaj wrote:
1) If the notification drawer is open and you forget to close it, it remains open even if you browse to some other page.
That was by design, you could argue against it.
If you go to : http://www.patternfly.org/pattern-library/communication/notification-drawer/notification-drawer-horizontal-nav.html
and open the notification bar and then refresh, you will notice that the notification bar is hidden.
2) Notification drawer has a weird placement, as in its far from its icon(i am not sure why that is happening).
Afair this happens in patternfly too.
Nope, it doesnt. we have it(right: 0) set in our css. If you make it right:auto then it should work just fine.
Updated by Ohad Levy over 6 years ago
- File drawer.png drawer.png added
Rahul Bajaj wrote:
Ohad Levy wrote:
Rahul Bajaj wrote:
1) If the notification drawer is open and you forget to close it, it remains open even if you browse to some other page.
That was by design, you could argue against it.
If you go to : http://www.patternfly.org/pattern-library/communication/notification-drawer/notification-drawer-horizontal-nav.html
and open the notification bar and then refresh, you will notice that the notification bar is hidden.
we have some pages that might refresh the page (e.g. dashboard).
I don't have a strong opinion on this feature, I suggest opening a PR and asking Rox for her feedback.
2) Notification drawer has a weird placement, as in its far from its icon(i am not sure why that is happening).
Afair this happens in patternfly too.
Nope, it doesnt. we have it(right: 0) set in our css. If you make it right:auto then it should work just fine.
right:auto does not work as expected for me, if I change it then the notification drawer is partly hidden to the right and i have to scroll (see attachment)
Updated by Rahul Bajaj over 6 years ago
Ohad Levy wrote:
Rahul Bajaj wrote:
Ohad Levy wrote:
Rahul Bajaj wrote:
1) If the notification drawer is open and you forget to close it, it remains open even if you browse to some other page.
That was by design, you could argue against it.
If you go to : http://www.patternfly.org/pattern-library/communication/notification-drawer/notification-drawer-horizontal-nav.html
and open the notification bar and then refresh, you will notice that the notification bar is hidden.we have some pages that might refresh the page (e.g. dashboard).
I don't have a strong opinion on this feature, I suggest opening a PR and asking Rox for her feedback.
I shall do that :)
2) Notification drawer has a weird placement, as in its far from its icon(i am not sure why that is happening).
Afair this happens in patternfly too.
Nope, it doesnt. we have it(right: 0) set in our css. If you make it right:auto then it should work just fine.
right:auto does not work as expected for me, if I change it then the notification drawer is partly hidden to the right and i have to scroll (see attachment)
Ah! i see, then may be the correct fix to my problem[attachment] would be making the notification bar's position:relative in the '#notifications_container' class
Updated by Rahul Bajaj over 6 years ago
- Target version changed from 1.17.0-RC1 to 1.15.6
Updated by Rahul Bajaj over 6 years ago
- Assignee deleted (
Rahul Bajaj) - Target version deleted (
1.15.6)