Fix mobile menus
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F15533217: D9086.diff
	Wed, Apr 23, 6:46 PM

	F15482740: D9086.diff
	Wed, Apr 9, 7:36 AM

	F15463533: D9086.id21590.diff
	Wed, Apr 2, 12:34 AM

	F15462436: D9086.id21593.diff
	Tue, Apr 1, 3:08 PM

	F15461370: D9086.id.diff
	Tue, Apr 1, 7:52 AM

	F15459006: D9086.diff
	Mon, Mar 31, 9:05 AM

	F15419453: D9086.id21590.diff
	Mar 21 2025, 6:11 AM

	F15414283: D9086.id21593.diff
	Mar 19 2025, 11:58 PM

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