Page MenuHomePhabricator

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

Authored by epriestley on Feb 20 2017, 8:51 PM.
Tags
None
Referenced Files
F15416292: D17388.id41811.diff
Thu, Mar 20, 10:24 AM
F15411547: D17388.id41811.diff
Wed, Mar 19, 9:26 AM
F15407731: D17388.id.diff
Tue, Mar 18, 7:12 PM
F15393986: D17388.id41811.diff
Sat, Mar 15, 11:18 PM
F15374777: D17388.id41811.diff
Wed, Mar 12, 8:17 PM
F15336275: D17388.diff
Sat, Mar 8, 6:36 PM
F15335340: D17388.id41811.diff
Sat, Mar 8, 3:06 PM
Unknown Object (File)
Feb 17 2025, 1:56 AM
Subscribers
None

Details

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
Branch
notif1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15708
Build 20750: Run Core Tests
Build 20749: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Feb 20 2017, 8:52 PM
This revision was automatically updated to reflect the committed changes.