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

	F13123100: D15719.diff
	Tue, Apr 30, 8:18 AM

	Unknown Object (File)
	Thu, Apr 25, 3:31 AM

	Unknown Object (File)
	Tue, Apr 16, 12:25 AM

	Unknown Object (File)
	Mon, Apr 15, 5:06 PM

	Unknown Object (File)
	Fri, Apr 12, 3:19 PM

	Unknown Object (File)
	Fri, Apr 5, 9:50 PM

	Unknown Object (File)
	Mar 31 2024, 9:32 AM

	Unknown Object (File)
	Feb 21 2024, 7:26 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...