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

	F15501344: D15719.id37878.diff
	Sun, Apr 13, 9:28 PM

	F15483894: D15719.diff
	Wed, Apr 9, 3:12 PM

	F15478037: D15719.id37878.diff
	Mon, Apr 7, 7:58 PM

	F15393215: D15719.id37880.diff
	Sat, Mar 15, 8:27 PM

	F15386812: D15719.id.diff
	Sat, Mar 15, 1:19 AM

	Unknown Object (File)
	Feb 16 2025, 4:04 PM

	Unknown Object (File)
	Feb 9 2025, 1:18 AM

	Unknown Object (File)
	Feb 9 2025, 1: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...