Remove text-rendering CSS rule
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13147817: D10940.diff
	Fri, May 3, 5:30 PM

	F13140789: D10940.diff
	Fri, May 3, 3:57 AM

	Unknown Object (File)
	Mon, Apr 29, 4:06 PM

	Unknown Object (File)
	Wed, Apr 24, 11:54 PM

	Unknown Object (File)
	Thu, Apr 11, 8:10 AM

	Unknown Object (File)
	Mar 31 2024, 2:41 PM

	Unknown Object (File)
	Mar 31 2024, 6:29 AM

	Unknown Object (File)
	Mar 27 2024, 5:23 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