revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-18 21:02:41 +01:00
Code Issues Releases Wiki Activity

Fix changeset layout in Modern Firefox

Browse source 
Summary: Looks like a rule for old Firefox is causing layout issues in new Firefox. Prefer new Firefox. Fixes T4987

Test Plan:
Tested the current version of Firefox.

Bug:

{F158209}

Reviewers: btrahan, epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T4987

Differential Revision: https://secure.phabricator.com/D9240
This commit is contained in:
Chad Little 2014-05-21 13:45:02 -07:00
parent a9d6a2f02d
commit 2ad501873a
2 changed files with 3 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
resources/celerity/map.php
View file

@ -10,7 +10,7 @@ return array(
'core.pkg.css' => '3445a3a7', 'core.pkg.css' => '3445a3a7',
'core.pkg.js' => 'ab0d6d3d', 'core.pkg.js' => 'ab0d6d3d',
'darkconsole.pkg.js' => 'ca8671ce', 'darkconsole.pkg.js' => 'ca8671ce',
'differential.pkg.css' => '382ca868', 'differential.pkg.css' => 'fbf57382',
'differential.pkg.js' => '68d225fb', 'differential.pkg.js' => '68d225fb',
'diffusion.pkg.css' => '3783278d', 'diffusion.pkg.css' => '3783278d',
'diffusion.pkg.js' => '077e3ad0', 'diffusion.pkg.js' => '077e3ad0',
@ -55,7 +55,7 @@ return array(
'rsrc/css/application/dashboard/dashboard.css' => 'c1d7f80b', 'rsrc/css/application/dashboard/dashboard.css' => 'c1d7f80b',
'rsrc/css/application/diff/inline-comment-summary.css' => '8cfd34e8', 'rsrc/css/application/diff/inline-comment-summary.css' => '8cfd34e8',
'rsrc/css/application/differential/add-comment.css' => 'c478bcaa', 'rsrc/css/application/differential/add-comment.css' => 'c478bcaa',
'rsrc/css/application/differential/changeset-view.css' => 'c45747f0', 'rsrc/css/application/differential/changeset-view.css' => 'f234b888',
'rsrc/css/application/differential/core.css' => '7ac3cabc', 'rsrc/css/application/differential/core.css' => '7ac3cabc',
'rsrc/css/application/differential/results-table.css' => '239924f9', 'rsrc/css/application/differential/results-table.css' => '239924f9',
'rsrc/css/application/differential/revision-comment.css' => '48186045', 'rsrc/css/application/differential/revision-comment.css' => '48186045',
@ -509,7 +509,7 @@ return array(
'conpherence-notification-css' => '403cf598', 'conpherence-notification-css' => '403cf598',
'conpherence-update-css' => '1099a660', 'conpherence-update-css' => '1099a660',
'conpherence-widget-pane-css' => 'bf275a6c', 'conpherence-widget-pane-css' => 'bf275a6c',
'differential-changeset-view-css' => 'c45747f0', 'differential-changeset-view-css' => 'f234b888',
'differential-core-view-css' => '7ac3cabc', 'differential-core-view-css' => '7ac3cabc',
'differential-inline-comment-editor' => 'f2441746', 'differential-inline-comment-editor' => 'f2441746',
'differential-results-table-css' => '239924f9', 'differential-results-table-css' => '239924f9',

11
webroot/rsrc/css/application/differential/changeset-view.css
View file

@ -41,17 +41,6 @@
} }
.differential-diff td { .differential-diff td {
/*
Disable ligatures in Firefox. Firefox 3 has fancypants ligature support, but
it gets applied to monospaced fonts, which sucks because it means that the
"fi" ligature only takes up one character, e.g. It's probably the font's
fault that it even specifies ligatures (seriously, what the hell?) but
that's hard to fix and this is "easy" to "fix": custom letter spacing
disables ligatures, as long as it's at least 0.008333-repeating pixels of
custom letter spacing. I have no idea where this number comes from, but note
that .83333.. = 5/6. -epriestley
*/
letter-spacing: 0.0083334px;
vertical-align: top; vertical-align: top;
white-space: pre-wrap; white-space: pre-wrap;
word-wrap: break-word; word-wrap: break-word;