Paths

Table of Contentst

Differential D14036

Scroll on overflow of property list (mobile)
ClosedPublic
Actions

Authored by chad on Sep 2 2015, 1:31 AM.

Details

Reviewers

epriestley

Maniphest Tasks

T9314: Legalpad document title overflows screen

Commits

Restricted Diffusion Commit
rP29399849c0ed: Scroll on overflow of property list (mobile)

Summary

Fixes T9314. Functionally phui-status-list should get moved off a table, but that's another day. This catches many other possible issues.

Test Plan

Review changes on a narrow browser.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 33935.Sep 2 2015, 1:31 AM

chad retitled this revision from to Scroll on overflow of property list (mobile).

chad updated this object.

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

chad added a reviewer: epriestley.

chad added a task: T9314: Legalpad document title overflows screen.

Herald added a subscriber: Korvin. · View Herald TranscriptSep 2 2015, 1:31 AM

Not auto? This will always render a scrollbar, right? Even if the text fits? Should probably be auto?

This revision is now accepted and ready to land.Sep 3 2015, 2:10 PM

Oh, I didn't notice, but I built it on a trackpad so it probably did the auto hide stuff.

Ah yeah.

Closed by commit rP29399849c0ed: Scroll on overflow of property list (mobile) (authored by chad, committed by chad). · Explain WhySep 3 2015, 6:06 PM

This revision was automatically updated to reflect the committed changes.

• cyn0228.ch added a task: T5375: Tokenizers fail in Firefox on Android.Sep 7 2015, 10:42 PM

chad removed a task: T5375: Tokenizers fail in Firefox on Android.Sep 7 2015, 10:45 PM

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

6 lines

webroot/

rsrc/

css/

phui/

phui-property-list-view.css

1 line

Diff 33977

View Options

resources/celerity/map.php