Page MenuHomePhabricator

Break all mobile form rendering
AbandonedPublic

Authored by chad on Apr 17 2017, 11:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 23 2024, 8:56 AM
Unknown Object (File)
Feb 11 2024, 9:44 AM
Unknown Object (File)
Jan 20 2024, 5:47 PM
Unknown Object (File)
Jan 9 2024, 4:36 PM
Unknown Object (File)
Jan 5 2024, 5:47 PM
Unknown Object (File)
Jan 1 2024, 5:57 PM
Unknown Object (File)
Jan 1 2024, 5:57 PM
Unknown Object (File)
Dec 28 2023, 7:00 PM
Subscribers

Details

Reviewers
epriestley
Summary

I'm not sure I want to ship this, but bumps font-size up on mobile devices.

Test Plan

Lots of different forms on desktop and mobile

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16540
Build 22038: Run Core Tests
Build 22037: arc lint + arc unit

Event Timeline

The zooming is soooooo awful, did this turn it back on on purpose or was that just collateral damage from things happening out of order?

Seems basically fine except for zooming (which is the worst)

webroot/rsrc/css/phui/phui-form.css
119–124

One of these is busted on multi-tokenizer inputs:

Screen Shot 2017-04-17 at 5.07.36 PM.png (312×680 px, 39 KB)

This seems like a lot of work for only a small theoretical benefit