Fix mobile menus
ClosedPublic
Actions

Authored by chad on May 13 2014, 2:05 AM.

Tags

None

Referenced Files

	F13058901: D9086.id.diff
	Fri, Apr 19, 3:27 PM

	Unknown Object (File)
	Thu, Apr 11, 8:54 AM

	Unknown Object (File)
	Tue, Apr 2, 9:46 AM

	Unknown Object (File)
	Mon, Apr 1, 6:46 AM

	Unknown Object (File)
	Sun, Mar 24, 4:42 AM

	Unknown Object (File)
	Fri, Mar 22, 8:07 AM

	Unknown Object (File)
	Feb 11 2024, 2:02 PM

	Unknown Object (File)
	Jan 20 2024, 5:45 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rP5b31a17c96e0: Fix mobile menus

Summary

The multiple icons are causing layout issues. This hides the unsused div (the real icon gets appended). We should probably clean this up next redesign.

Test Plan

test mobile layouts in Chrome, test main menu and application menu. Go to Differential, check those menus to.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

chad updated this revision to Diff 21590.May 13 2014, 2:05 AM

chad retitled this revision from to Fix mobile menus.

chad updated this object.

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

chad added a reviewer: epriestley.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptMay 13 2014, 2:05 AM

Harbormaster completed remote builds in B414: Diff 21590.May 13 2014, 2:06 AM

epriestley accepted this revision.May 13 2014, 2:40 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 13 2014, 2:40 AM

Closed by commit rP5b31a17c96e0 (authored by @chad).

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

application/

base/

main-menu-view.css

4 lines

Diff 21593

View Options

resources/celerity/map.php