Fix crumb alignment with new notification layout
ClosedPublic
Actions

Authored by chad on Dec 9 2014, 4:55 AM.

Tags

None

Referenced Files

	F15472331: D10949.id26295.diff
	Sat, Apr 5, 12:41 PM

	F15401802: D10949.diff
	Mon, Mar 17, 7:19 PM

	F15331488: D10949.diff
	Mar 7 2025, 10:36 AM

	Unknown Object (File)
	Feb 26 2025, 2:46 PM

	Unknown Object (File)
	Feb 8 2025, 12:16 AM

	Unknown Object (File)
	Feb 8 2025, 12:16 AM

	Unknown Object (File)
	Feb 8 2025, 12:16 AM

	Unknown Object (File)
	Feb 8 2025, 12:16 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley
btrahan

Commits

Restricted Diffusion Commit
rPe3f99947ac80: Fix crumb alignment with new notification layout

Summary

Crumbs misalign because the hit area on the icons was 2px too large.

Test Plan

Test on pages with crumbs. IE, FF, Chrome...

Diff Detail

Repository

rP Phabricator

Branch

derpies

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3201
Build 3207: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 26294.Dec 9 2014, 4:55 AM

chad retitled this revision from to Fix crumb alignment with new notification layout.

chad updated this object.

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

chad added reviewers: epriestley, btrahan.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptDec 9 2014, 4:55 AM

Closed by commit rPe3f99947ac80: Fix crumb alignment with new notification layout (authored by chad, committed by chad). · Explain WhyDec 9 2014, 5:04 AM

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/

application/

base/

main-menu-view.css

2 lines

Diff 26294

View Options

resources/celerity/map.php