Page MenuHomePhabricator
Differential D19142

Fix some of the most egregious errors in Harbormaster log paging
ClosedPublic
Actions

Authored by epriestley on Feb 27 2018, 1:49 AM.
Tags
None
Referenced Files
F18813494: D19142.diff
Mon, Oct 20, 4:54 PM
F18811643: D19142.id45862.diff
Mon, Oct 20, 4:05 AM
F18794620: D19142.id45850.diff
Fri, Oct 17, 12:07 AM
F18794619: D19142.id45862.diff
Fri, Oct 17, 12:07 AM
F18794403: D19142.diff
Thu, Oct 16, 11:35 PM
F18634017: D19142.diff
Sep 16 2025, 9:24 PM
F18619222: D19142.diff
Sep 15 2025, 2:00 AM
F18566661: D19142.id.diff
Sep 9 2025, 2:48 PM
View All Files
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
T13088: Plans: Harbormaster UI usability and interconnectedness
Commits
rPf450c6c55bfd: Fix some of the most egregious errors in Harbormaster log paging
Summary

Depends on D19141. Ref T13088. Some of the fundamental log behaviors like "loading the correct rows" are now a bit better behaved.

The UI is a little less garbage, too.

Test Plan

Viewed some logs and loaded more context by clicking the buttons.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 27 2018, 1:49 AM
Owners added a subscriber: Restricted Owners Package.Feb 27 2018, 1:49 AM
Harbormaster completed remote builds in B19661: Diff 45850.Feb 27 2018, 1:51 AM
epriestley requested review of this revision.Feb 27 2018, 1:51 AM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 27 2018, 1:59 AM
Closed by commit rPf450c6c55bfd: Fix some of the most egregious errors in Harbormaster log paging (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
14 lines
src/
applications/
harbormaster/
controller/
196 lines
webroot/
rsrc/
css/
application/
harbormaster/
64 lines
js/
application/
harbormaster/
4 lines

Diff 45862

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/harbormaster/controller/HarbormasterBuildLogRenderController.php

Loading...
View Options

webroot/rsrc/css/application/harbormaster/harbormaster.css

Loading...
View Options

webroot/rsrc/js/application/harbormaster/behavior-harbormaster-log.js

Loading...
Phabricator · Built by Phacility