Fix crumb alignment with new notification layout
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F14063475: D10949.diff
	Mon, Nov 18, 6:41 PM

	F14052136: D10949.diff
	Fri, Nov 15, 6:41 AM

	F14040120: D10949.id26295.diff
	Mon, Nov 11, 7:30 AM

	F14037882: D10949.diff
	Sun, Nov 10, 7:47 PM

	F14023474: D10949.diff
	Thu, Nov 7, 2:12 AM

	F14017802: D10949.id26295.diff
	Mon, Nov 4, 11:02 PM

	F14004756: D10949.id26294.diff
	Sun, Oct 27, 2:17 AM

	Unknown Object (File)
	Oct 9 2024, 5:13 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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 26295

View Options

resources/celerity/map.php