@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
When the user clicks the notification menu, hide it
Summary: Fixes T7905.
Test Plan: With Quicksand, clicked links and empty space in the notification menu.
Reviewers: btrahan
Reviewed By: btrahan
Subscribers: epriestley
Maniphest Tasks: T7905
Differential Revision: https://secure.phabricator.com/D12541
+16
-14
+13
-13
resources/celerity/map.php
+13
-13
resources/celerity/map.php
···
8
8
return array(
9
9
'names' => array(
10
10
'core.pkg.css' => '9a9b59ca',
11
-
'core.pkg.js' => '6f7446de',
11
+
'core.pkg.js' => '59d01bb7',
12
12
'darkconsole.pkg.js' => '8ab24e01',
13
13
'differential.pkg.css' => '3500921f',
14
14
'differential.pkg.js' => '890046d3',
···
350
350
'rsrc/image/texture/table_header_hover.png' => '038ec3b9',
351
351
'rsrc/image/texture/table_header_tall.png' => 'd56b434f',
352
352
'rsrc/js/application/aphlict/Aphlict.js' => '30a6303c',
353
-
'rsrc/js/application/aphlict/behavior-aphlict-dropdown.js' => 'ee37f73a',
353
+
'rsrc/js/application/aphlict/behavior-aphlict-dropdown.js' => '0323afdd',
354
354
'rsrc/js/application/aphlict/behavior-aphlict-listen.js' => 'b1a59974',
355
355
'rsrc/js/application/aphlict/behavior-aphlict-status.js' => 'ea681761',
356
356
'rsrc/js/application/auth/behavior-persona-login.js' => '9414ff18',
···
550
550
'inline-comment-summary-css' => 'eb5f8e8c',
551
551
'javelin-aphlict' => '30a6303c',
552
552
'javelin-behavior' => '61cbc29a',
553
-
'javelin-behavior-aphlict-dropdown' => 'ee37f73a',
553
+
'javelin-behavior-aphlict-dropdown' => '0323afdd',
554
554
'javelin-behavior-aphlict-listen' => 'b1a59974',
555
555
'javelin-behavior-aphlict-status' => 'ea681761',
556
556
'javelin-behavior-aphront-basic-tokenizer' => 'b3a4b884',
···
844
844
'requires' => array(
845
845
'029a133d' => array(
846
846
'aphront-dialog-view-css',
847
+
),
848
+
'0323afdd' => array(
849
+
'javelin-behavior',
850
+
'javelin-request',
851
+
'javelin-stratcom',
852
+
'javelin-vector',
853
+
'javelin-dom',
854
+
'javelin-uri',
855
+
'javelin-behavior-device',
856
+
'phabricator-title',
847
857
),
848
858
'048330fa' => array(
849
859
'javelin-behavior',
···
1939
1949
'javelin-dom',
1940
1950
'javelin-stratcom',
1941
1951
'javelin-vector',
1942
-
),
1943
-
'ee37f73a' => array(
1944
-
'javelin-behavior',
1945
-
'javelin-request',
1946
-
'javelin-stratcom',
1947
-
'javelin-vector',
1948
-
'javelin-dom',
1949
-
'javelin-uri',
1950
-
'javelin-behavior-device',
1951
-
'phabricator-title',
1952
1952
),
1953
1953
'efe49472' => array(
1954
1954
'javelin-install',
+3
-1
webroot/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js
+3
-1
webroot/rsrc/js/application/aphlict/behavior-aphlict-dropdown.js
···
138
138
}
139
139
140
140
if (e.getNode('tag:a')) {
141
-
// User clicked a link, just follow the link.
141
+
// User clicked a link. Hide the menu, then follow the link.
142
+
set_visible(null);
142
143
return;
143
144
}
144
145
···
154
155
if (href) {
155
156
JX.$U(href).go();
156
157
e.kill();
158
+
set_visible(null);
157
159
}
158
160
});
159
161