Center Pager buttons
ClosedPublic
Actions

Authored by chad on Feb 22 2017, 4:29 PM.

Tags

None

Referenced Files

	F19803681: D17394.diff
	Mon, Mar 2, 1:16 AM

	F19800654: D17394.diff
	Sat, Feb 28, 5:02 PM

	F19673807: D17394.id41823.diff
	Sun, Feb 8, 6:43 AM

	F19538744: D17394.id41823.diff
	Jan 21 2026, 4:41 PM

	F19534922: D17394.id41823.diff
	Jan 21 2026, 3:24 AM

	F19059801: D17394.diff
	Nov 29 2025, 6:51 AM

	F18849544: D17394.id.diff
	Oct 30 2025, 10:35 AM

	F18844806: D17394.diff
	Oct 29 2025, 7:43 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Maniphest Tasks

T12305: Persistent chat overlaps various "next" buttons

Commits

rPaaa81b48349d: Center Pager buttons

Summary

Fixes T12305. Centers the buttons, which I prefer anyways.

Test Plan

Review buttons, now centered.

Diff Detail

Repository

rP Phabricator

Branch

pager (branched from master)

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15719
Build 20770: Run Core Tests
Build 20769: arc lint + arc unit

Event Timeline

chad created this revision.Feb 22 2017, 4:29 PM

Herald added a subscriber: Korvin. · View Herald TranscriptFeb 22 2017, 4:29 PM

Harbormaster completed remote builds in B15719: Diff 41822.Feb 22 2017, 4:30 PM

I'm having trouble accepting this revision because I can't find the button to submit the comment? Where is it??

Screen Shot 2017-02-22 at 8.29.56 AM.png (1×1 px, 241 KB)

This revision is now accepted and ready to land.Feb 22 2017, 4:30 PM

I actually would prefer that comment box to not be full width, like in Maniphest, etc. I think Differential/Audit it's inconsistently wide.

Closed by commit rPaaa81b48349d: Center Pager buttons (authored by chad, committed by chad). · Explain WhyFeb 22 2017, 4:34 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

phui/

phui-pager.css

2 lines

Diff 41822

View Options

resources/celerity/map.php