Homec4science

Settings - upgrade monospace font regexp to support '.'

Authored by Bob Trahan <btrahan@phacility.com> on Aug 6 2014, 22:53.

Description

Settings - upgrade monospace font regexp to support '.'

Summary: this data is a little weird since its user-entered and we need to put it in a web page un-escaped for the font to load correctly. Ergo, we use a regex to make the input safe / sane, and said regex needs to support a '.'. Fixes T5810.

Test Plan: added Fixedsys Excelsior 3.01 to my system and was able to set my preference and get the new font

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: dereckson, epriestley, Korvin

Maniphest Tasks: T5810

Differential Revision: https://secure.phabricator.com/D10163

Details

Committed
Bob Trahan <btrahan@phacility.com>Aug 6 2014, 22:53
Pushed
aubortJan 31 2017, 17:16
Parents
rPH9c1c4bb5ae71: Move artifacts and build target messages into tabs
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPH20a65b21eba9: Settings - upgrade monospace font regexp to support '.' (authored by Bob Trahan <btrahan@phacility.com>).Aug 6 2014, 22:53