Remove text-rendering CSS rule
ClosedPublic
Actions

Authored by chad on Dec 8 2014, 1:46 AM.

Tags

None

Referenced Files

	F15399018: D10940.id.diff
	Mon, Mar 17, 2:28 AM

	F15378265: D10940.id26276.diff
	Mar 13 2025, 1:31 PM

	F15378264: D10940.id26275.diff
	Mar 13 2025, 1:31 PM

	F15378263: D10940.id.diff
	Mar 13 2025, 1:31 PM

	F15378262: D10940.diff
	Mar 13 2025, 1:30 PM

	F15378197: D10940.id26275.diff
	Mar 13 2025, 1:27 PM

	F15343131: D10940.id26276.diff
	Mar 9 2025, 10:13 PM

	F15343129: D10940.id26275.diff
	Mar 9 2025, 10:13 PM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rP46b775820a50: Remove text-rendering CSS rule

Summary

This rule is making text actually more difficult to read on Windows, as well as is broken on Win/Chrome, and evidentally has performance issues on mobile browsers.

Test Plan

Turn it off, can still read web page

Diff Detail

Repository

rP Phabricator

Branch

text-legibility

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 3192
Build 3198: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 26275.Dec 8 2014, 1:46 AM

chad retitled this revision from to Remove text-rendering CSS rule.

chad updated this object.

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

chad added a reviewer: epriestley.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptDec 8 2014, 1:46 AM

epriestley accepted this revision.Dec 8 2014, 4:04 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Dec 8 2014, 4:04 AM

Closed by commit rP46b775820a50: Remove text-rendering CSS rule (authored by chad, committed by chad). · Explain WhyDec 8 2014, 4:08 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/

core/

core.css

1 line

Diff 26275

View Options

resources/celerity/map.php