Differential D15719

Add coverage tooltips in Diffusion file browse mode
ClosedPublic
Actions

Authored by epriestley on Apr 15 2016, 12:39 PM.

Tags

None

Referenced Files

	F13236181: D15719.id37878.diff
	Tue, May 21, 8:39 AM

	F13234750: D15719.diff
	Tue, May 21, 3:43 AM

	F13223477: D15719.id37880.diff
	Sun, May 19, 4:38 AM

	F13212705: D15719.diff
	Fri, May 17, 6:59 AM

	F13197635: D15719.diff
	Mon, May 13, 12:21 AM

	Unknown Object (File)
	Fri, May 3, 10:05 AM

	Unknown Object (File)
	Thu, May 2, 12:41 PM

	Unknown Object (File)
	Tue, Apr 30, 8:18 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10816: File-level coverage doesn't have tooltips

Commits

rP053400289469: Add coverage tooltips in Diffusion file browse mode

Summary

Fixes T10816. The way these work is a little unusual since these chunks of file-rendering code are unusuall performance-sensitive, so the Differential version doesn't adapt directly to Diffusion. Both can possibly be unified at some point in the future, although they do slightly different things.

Test Plan

coverage.png (614×1 px, 138 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37878.Apr 15 2016, 12:39 PM

epriestley retitled this revision from to Add coverage tooltips in Diffusion file browse mode.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T10816: File-level coverage doesn't have tooltips.

epriestley mentioned this in T10816: File-level coverage doesn't have tooltips.Apr 15 2016, 12:45 PM

chad accepted this revision.Apr 15 2016, 1:50 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Apr 15 2016, 1:50 PM

Closed by commit rP053400289469: Add coverage tooltips in Diffusion file browse mode (authored by epriestley, committed by epriestley). · Explain WhyApr 15 2016, 1:59 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

8 lines

src/

applications/

diffusion/

controller/

DiffusionBrowseController.php

14 lines

webroot/

rsrc/

js/

application/

diffusion/

behavior-diffusion-browse-file.js

47 lines

Diff 37880

resources/celerity/map.php

Loading...

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...

webroot/rsrc/js/application/diffusion/behavior-diffusion-browse-file.js

Loading...