Use selected button state on blame button
ClosedPublic
Actions

Authored by chad on Aug 30 2017, 5:24 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Dec 19, 7:38 PM

	Unknown Object (File)
	Fri, Dec 13, 8:01 PM

	Unknown Object (File)
	Thu, Dec 12, 8:43 AM

	Unknown Object (File)
	Sun, Dec 8, 10:30 AM

	Unknown Object (File)
	Wed, Dec 4, 3:20 AM

	Unknown Object (File)
	Tue, Dec 3, 2:41 PM

	Unknown Object (File)
	Sat, Nov 30, 9:08 AM

	Unknown Object (File)
	Sat, Nov 30, 1:08 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rP67c658a7ed87: Use selected button state on blame button

Summary

Visually selects the button if blame is on.

Test Plan

Turn blame on and off in Diffusion on a file.

Diff Detail

Repository

rP Phabricator

Branch

blame-button (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 18223
Build 24508: Run Core Tests
Build 24507: arc lint + arc unit

Event Timeline

chad created this revision.Aug 30 2017, 5:24 PM

Harbormaster completed remote builds in B18223: Diff 44438.Aug 30 2017, 5:24 PM

Herald added a subscriber: Korvin. · View Herald TranscriptAug 30 2017, 5:24 PM

Harbormaster completed remote builds in B18223: Diff 44438.Aug 30 2017, 5:25 PM

epriestley accepted this revision.Aug 30 2017, 5:45 PM

This revision is now accepted and ready to land.Aug 30 2017, 5:45 PM

Closed by commit rP67c658a7ed87: Use selected button state on blame button (authored by Chad Little <chad@phacility.com>, committed by chad). · Explain WhyAug 30 2017, 6:33 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

controller/

DiffusionBrowseController.php

1 line

Diff 44438

View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Use selected button state on blame buttonClosedPublicActions