Remove text-rendering CSS rule
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F14005312: D10940.id26276.diff
	Sun, Oct 27, 11:13 AM

	F13995072: D10940.id26276.diff
	Wed, Oct 23, 10:28 AM

	F13974452: D10940.id.diff
	Oct 18 2024, 5:37 AM

	F13958027: D10940.id26275.diff
	Oct 14 2024, 12:30 PM

	F13955197: D10940.id26275.diff
	Oct 14 2024, 12:28 AM

	Unknown Object (File)
	Oct 9 2024, 3:17 AM

	Unknown Object (File)
	Oct 4 2024, 12:48 PM

	Unknown Object (File)
	Oct 1 2024, 1:52 AM

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