Page MenuHomePhabricator
Differential D17388

Fix bad JS rendering in "Allow Desktop Notifications" workflow
ClosedPublic
Actions

Authored by epriestley on Feb 20 2017, 8:51 PM.
Tags
None
Referenced Files
F19046329: D17388.id.diff
Nov 27 2025, 10:11 AM
F19041308: D17388.diff
Nov 26 2025, 3:34 PM
F18857401: D17388.id.diff
Nov 1 2025, 3:03 PM
F18843105: D17388.diff
Oct 28 2025, 8:59 PM
F18834708: D17388.id.diff
Oct 26 2025, 10:19 AM
F18831807: D17388.diff
Oct 25 2025, 3:25 PM
F18829860: D17388.id41811.diff
Oct 25 2025, 1:41 AM
F18783885: D17388.id41812.diff
Oct 13 2025, 10:26 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
rPab9c1b73b532: Fix bad JS rendering in "Allow Desktop Notifications" workflow
Summary

See downstream https://phabricator.kde.org/T5404. This code was doing some .firstChild shenanigans which didn't survive some UI refactoring.

This whole UI is a little iffy but just unbreak it for now.

Test Plan

Allowed and rejected desktop notifications, got largely reasonable UI rendering.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 20 2017, 8:51 PM
Harbormaster completed remote builds in B15708: Diff 41811.Feb 20 2017, 8:52 PM
chad accepted this revision.Feb 20 2017, 8:52 PM
This revision is now accepted and ready to land.Feb 20 2017, 8:52 PM
Closed by commit rPab9c1b73b532: Fix bad JS rendering in "Allow Desktop Notifications" workflow (authored by epriestley, committed by epriestley). · Explain WhyFeb 20 2017, 8:55 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
18 lines
src/
applications/
settings/
panel/
12 lines
webroot/
rsrc/
js/
application/
aphlict/
16 lines

Diff 41812

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/settings/panel/PhabricatorDesktopNotificationsSettingsPanel.php

Loading...
View Options

webroot/rsrc/js/application/aphlict/behavior-desktop-notifications-control.js

Loading...
Phabricator · Built by Phacility