Page MenuHomePhabricator

D11597.diff
No OneTemporary

D11597.diff

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -7,10 +7,10 @@
*/
return array(
'names' => array(
- 'core.pkg.css' => '3fa78a8d',
+ 'core.pkg.css' => 'dc3f102a',
'core.pkg.js' => '23d653bb',
'darkconsole.pkg.js' => '8ab24e01',
- 'differential.pkg.css' => '14f0fa92',
+ 'differential.pkg.css' => '4c3242f8',
'differential.pkg.js' => '7b5a4aa4',
'diffusion.pkg.css' => '591664fa',
'diffusion.pkg.js' => 'bfc0737b',
@@ -56,7 +56,7 @@
'rsrc/css/application/dashboard/dashboard.css' => 'c0062064',
'rsrc/css/application/diff/inline-comment-summary.css' => 'eb5f8e8c',
'rsrc/css/application/differential/add-comment.css' => 'c478bcaa',
- 'rsrc/css/application/differential/changeset-view.css' => '8244e5a8',
+ 'rsrc/css/application/differential/changeset-view.css' => 'a7781046',
'rsrc/css/application/differential/core.css' => '7ac3cabc',
'rsrc/css/application/differential/results-table.css' => '181aa9d9',
'rsrc/css/application/differential/revision-comment.css' => '48186045',
@@ -106,7 +106,7 @@
'rsrc/css/application/tokens/tokens.css' => '3d0f239e',
'rsrc/css/application/uiexample/example.css' => '528b19de',
'rsrc/css/core/core.css' => 'c8c5ecd2',
- 'rsrc/css/core/remarkup.css' => '07f22227',
+ 'rsrc/css/core/remarkup.css' => '2dbff225',
'rsrc/css/core/syntax.css' => '56c1ba38',
'rsrc/css/core/z-index.css' => '8239495e',
'rsrc/css/diviner/diviner-shared.css' => '38813222',
@@ -130,7 +130,7 @@
'rsrc/css/phui/phui-document.css' => '8240b0b1',
'rsrc/css/phui/phui-error-view.css' => '41518665',
'rsrc/css/phui/phui-feed-story.css' => 'c9f3a0b5',
- 'rsrc/css/phui/phui-fontkit.css' => 'bd111278',
+ 'rsrc/css/phui/phui-fontkit.css' => '4394f216',
'rsrc/css/phui/phui-form-view.css' => '8b78a986',
'rsrc/css/phui/phui-form.css' => 'f535f938',
'rsrc/css/phui/phui-header-view.css' => '083669db',
@@ -522,7 +522,7 @@
'conpherence-notification-css' => '04a6e10a',
'conpherence-update-css' => '1099a660',
'conpherence-widget-pane-css' => '3d575438',
- 'differential-changeset-view-css' => '8244e5a8',
+ 'differential-changeset-view-css' => 'a7781046',
'differential-core-view-css' => '7ac3cabc',
'differential-inline-comment-editor' => 'f2441746',
'differential-results-table-css' => '181aa9d9',
@@ -740,7 +740,7 @@
'phabricator-phtize' => 'd254d646',
'phabricator-prefab' => '72da38cc',
'phabricator-profile-css' => 'd8cb982b',
- 'phabricator-remarkup-css' => '07f22227',
+ 'phabricator-remarkup-css' => '2dbff225',
'phabricator-search-results-css' => 'f240504c',
'phabricator-shaped-request' => '7cbe244b',
'phabricator-side-menu-view-css' => '7e8c6341',
@@ -785,7 +785,7 @@
'phui-error-view-css' => '41518665',
'phui-feed-story-css' => 'c9f3a0b5',
'phui-font-icon-base-css' => '3dad2ae3',
- 'phui-fontkit-css' => 'bd111278',
+ 'phui-fontkit-css' => '4394f216',
'phui-form-css' => 'f535f938',
'phui-form-view-css' => '8b78a986',
'phui-header-view-css' => '083669db',
diff --git a/src/applications/config/option/PhabricatorSyntaxHighlightingConfigOptions.php b/src/applications/config/option/PhabricatorSyntaxHighlightingConfigOptions.php
--- a/src/applications/config/option/PhabricatorSyntaxHighlightingConfigOptions.php
+++ b/src/applications/config/option/PhabricatorSyntaxHighlightingConfigOptions.php
@@ -140,7 +140,7 @@
$this->newOption(
'style.monospace',
'string',
- '10px "Menlo", "Consolas", "Monaco", monospace')
+ null)
->setLocked(true)
->setSummary(
pht('Default monospace font.'))
@@ -151,7 +151,7 @@
$this->newOption(
'style.monospace.windows',
'string',
- '11px "Menlo", "Consolas", "Monaco", monospace')
+ null)
->setLocked(true)
->setSummary(
pht('Default monospace font for clients on Windows.'))
diff --git a/src/view/page/PhabricatorStandardPageView.php b/src/view/page/PhabricatorStandardPageView.php
--- a/src/view/page/PhabricatorStandardPageView.php
+++ b/src/view/page/PhabricatorStandardPageView.php
@@ -270,18 +270,31 @@
$response = CelerityAPI::getStaticResourceResponse();
+ $font_css = null;
+ if (!empty($monospaced)) {
+ $font_css = hsprintf(
+ '<style type="text/css">'.
+ '.PhabricatorMonospaced, '.
+ '.phabricator-remarkup .remarkup-code-block '.
+ '.remarkup-code { font: %s !important; } '.
+ '</style>', $monospaced);
+ }
+
+ $font_css_win = null;
+ if (!empty($monospaced_win)) {
+ $font_css_win = hsprintf(
+ '<style type="text/css">'.
+ '.platform-windows .PhabricatorMonospaced, '.
+ '.platform-windows .phabricator-remarkup '.
+ '.remarkup-code-block .remarkup-code { font: %s !important; }'.
+ '</style>', $monospaced_win);
+ }
+
return hsprintf(
- '%s<style type="text/css">'.
- '.PhabricatorMonospaced, '.
- '.phabricator-remarkup .remarkup-code-block '.
- '.remarkup-code { font: %s; } '.
- '.platform-windows .PhabricatorMonospaced, '.
- '.platform-windows .phabricator-remarkup '.
- '.remarkup-code-block .remarkup-code { font: %s; }'.
- '</style>%s',
+ '%s%s%s%s',
parent::getHead(),
- phutil_safe_html($monospaced),
- phutil_safe_html($monospaced_win),
+ $font_css,
+ $font_css_win,
$response->renderSingleResource('javelin-magical-init', 'phabricator'));
}
diff --git a/webroot/rsrc/css/application/differential/changeset-view.css b/webroot/rsrc/css/application/differential/changeset-view.css
--- a/webroot/rsrc/css/application/differential/changeset-view.css
+++ b/webroot/rsrc/css/application/differential/changeset-view.css
@@ -61,7 +61,6 @@
color: {$bluetext};
cursor: pointer;
border-right: 1px solid {$thinblueborder};
- font-size: 11px;
overflow: hidden;
-moz-user-select: -moz-none;
diff --git a/webroot/rsrc/css/core/remarkup.css b/webroot/rsrc/css/core/remarkup.css
--- a/webroot/rsrc/css/core/remarkup.css
+++ b/webroot/rsrc/css/core/remarkup.css
@@ -10,6 +10,21 @@
margin: 0 0 12px;
}
+.PhabricatorMonospaced,
+.phabricator-remarkup .remarkup-code-block .remarkup-code {
+ font: 10px "Menlo", "Consolas", "Monaco", monospace;
+}
+
+.platform-windows .PhabricatorMonospaced,
+.platform-windows .phabricator-remarkup .remarkup-code-block .remarkup-code {
+ font: 11px "Menlo", "Consolas", "Monaco", monospace;
+}
+
+.phabricator-remarkup .remarkup-code-block {
+ margin: 12px 0;
+ white-space: pre;
+}
+
.phabricator-remarkup .remarkup-code-header {
padding: 4px 1px;
font-size: 13px;
diff --git a/webroot/rsrc/css/phui/phui-fontkit.css b/webroot/rsrc/css/phui/phui-fontkit.css
--- a/webroot/rsrc/css/phui/phui-fontkit.css
+++ b/webroot/rsrc/css/phui/phui-fontkit.css
@@ -17,6 +17,21 @@
line-height: 18px;
}
+.PhabricatorMonospaced .phui-document-view .phui-font-source-sans,
+.phui-document-view .phui-font-source-sans .phabricator-remarkup
+ .remarkup-code-block .remarkup-code {
+ font-size: 11px;
+ line-height: 1.3em;
+}
+
+.platform-windows .PhabricatorMonospaced .phui-document-view
+ .phui-font-source-sans,
+.platform-windows .phui-document-view .phui-font-source-sans
+ .phabricator-remarkup .remarkup-code-block .remarkup-code {
+ font-size: 12px;
+ line-height: 1.3em;
+}
+
.phui-font-source-sans .phabricator-remarkup h2.remarkup-header {
padding: 8px 24px 8px 0;
border-bottom: 1px solid {$thinblueborder};

File Metadata

Mime Type
text/plain
Expires
Tue, Oct 22, 8:08 AM (3 w, 18 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6733529
Default Alt Text
D11597.diff (7 KB)

Event Timeline