Notification dropdown fixes
Summary: dropdown now displays properly and without flicker Test Plan: tested locally Reviewers: epriestley Reviewed By: epriestley CC: allenjohnashton, keebuhm, aran, Korvin Differential Revision: https://secure.phabricator.com/D2708
This commit is contained in:
@@ -758,7 +758,7 @@ celerity_register_resource_map(array(
|
||||
),
|
||||
'javelin-behavior-aphlict-dropdown' =>
|
||||
array(
|
||||
'uri' => '/res/d0025c08/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js',
|
||||
'uri' => '/res/40c65741/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js',
|
||||
'type' => 'js',
|
||||
'requires' =>
|
||||
array(
|
||||
|
||||
@@ -414,8 +414,9 @@ final class PhabricatorStandardPageView extends AphrontPageView {
|
||||
'div',
|
||||
array(
|
||||
'sigil' => 'aphlict-dropdown',
|
||||
'id' => 'phabricator-notification-dropdown',
|
||||
),
|
||||
'id' => 'phabricator-notification-dropdown',
|
||||
'style' => 'display: none',
|
||||
),
|
||||
'');
|
||||
}
|
||||
|
||||
|
||||
@@ -12,9 +12,6 @@ JX.behavior('aphlict-dropdown', function(config) {
|
||||
var indicator = JX.$('phabricator-notification-indicator');
|
||||
var visible = false;
|
||||
|
||||
|
||||
JX.DOM.hide(dropdown);
|
||||
|
||||
//populate panel
|
||||
(new JX.Request('/notification/panel/',
|
||||
function(response) {
|
||||
|
||||
Reference in New Issue
Block a user