Remove text-rendering CSS rule
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Jan 23, 6:55 PM

	Unknown Object (File)
	Thu, Jan 23, 6:54 PM

	Unknown Object (File)
	Thu, Jan 23, 6:54 PM

	Unknown Object (File)
	Thu, Jan 23, 6:54 PM

	Unknown Object (File)
	Tue, Jan 21, 8:58 PM

	Unknown Object (File)
	Sun, Jan 12, 11:22 AM

	Unknown Object (File)
	Dec 27 2024, 9:03 PM

	Unknown Object (File)
	Dec 25 2024, 10:59 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