Page MenuHomePhabricator
Files F15377988

D11666.id28076.diff
No OneTemporary
Actions

D11666.id28076.diff
View Options

diff --git a/resources/celerity/map.php b/resources/celerity/map.php
--- a/resources/celerity/map.php
+++ b/resources/celerity/map.php
@@ -10,7 +10,7 @@
'core.pkg.css' => '8be366b7',
'core.pkg.js' => '65e04767',
'darkconsole.pkg.js' => '8ab24e01',
- 'differential.pkg.css' => '8af45893',
+ 'differential.pkg.css' => '33dd3e78',
'differential.pkg.js' => '7b5a4aa4',
'diffusion.pkg.css' => '591664fa',
'diffusion.pkg.js' => 'bfc0737b',
@@ -53,7 +53,7 @@
'rsrc/css/application/contentsource/content-source-view.css' => '4b8b05d4',
'rsrc/css/application/countdown/timer.css' => '86b7b0a0',
'rsrc/css/application/dashboard/dashboard.css' => 'ec324e2a',
- 'rsrc/css/application/diff/inline-comment-summary.css' => '8cfd34e8',
+ 'rsrc/css/application/diff/inline-comment-summary.css' => '96d915b5',
'rsrc/css/application/differential/add-comment.css' => 'c478bcaa',
'rsrc/css/application/differential/changeset-view.css' => 'b2b71e76',
'rsrc/css/application/differential/core.css' => '7ac3cabc',
@@ -534,7 +534,7 @@
'herald-rule-editor' => '6e2de6f2',
'herald-test-css' => '778b008e',
'homepage-panel-css' => 'e34bf140',
- 'inline-comment-summary-css' => '8cfd34e8',
+ 'inline-comment-summary-css' => '96d915b5',
'javelin-aphlict' => '2be71d56',
'javelin-behavior' => '61cbc29a',
'javelin-behavior-aphlict-dropdown' => '335470d7',
diff --git a/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php b/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
--- a/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
+++ b/src/infrastructure/diff/view/PhabricatorInlineSummaryView.php
@@ -15,20 +15,7 @@
public function render() {
require_celerity_resource('inline-comment-summary-css');
- return hsprintf('%s%s', $this->renderHeader(), $this->renderTable());
- }
-
- private function renderHeader() {
- $icon = id(new PHUIIconView())
- ->setIconFont('fa-comment bluegrey msr');
-
- $header = phutil_tag_div(
- 'phabricator-inline-summary',
- array(
- $icon,
- pht('Inline Comments'),
- ));
- return $header;
+ return hsprintf('%s', $this->renderTable());
}
private function renderTable() {
@@ -69,7 +56,7 @@
if ($href) {
$icon = id(new PHUIIconView())
- ->setIconFont('fa-share white msr');
+ ->setIconFont('fa-share darkbluetext msr');
$lines = phutil_tag(
'a',
diff --git a/webroot/rsrc/css/application/diff/inline-comment-summary.css b/webroot/rsrc/css/application/diff/inline-comment-summary.css
--- a/webroot/rsrc/css/application/diff/inline-comment-summary.css
+++ b/webroot/rsrc/css/application/diff/inline-comment-summary.css
@@ -2,36 +2,39 @@
* @provides inline-comment-summary-css
*/
-.phabricator-inline-summary {
- color: {$bluetext};
- padding-top: 12px;
- margin-top: 12px;
- border-top: 1px solid {$thinblueborder};
-}
-
-.phabricator-remarkup + .phabricator-inline-summary {
- padding-top: 16px;
+.transaction-comment + .phabricator-inline-summary-table {
+ margin-top: 16px;
}
.phabricator-inline-summary-table {
- width: 100%;
+ border-collapse: separate;
+ border-spacing: 1px;
+ background: #BFCFDA;
+ min-width: 66.6%;
}
.phabricator-inline-summary-table th {
- padding: 6px 0px;
- color: {$lightgreytext};
+ font-weight: bold;
+ padding: 8px;
+ background: #fff;
+ color: {$darkbluetext};
}
.phabricator-inline-summary-table td {
- padding: 0px 4px 6px;
+ padding: 4px 8px;
white-space: nowrap;
color: {$darkbluetext};
+ background: white;
}
.phabricator-inline-summary-table td.inline-line-number {
- padding-left: 0px;
+ padding: 0;
width: 100px;
white-space: nowrap;
+ background: #F8F9FC;
+ font-family: "Menlo", "Consolas", monospace;
+ font-size: 11px;
+ color: {$bluetext};
}
.phabricator-inline-summary-table td.inline-which-diff {
@@ -49,17 +52,15 @@
target (by highlighting the entire cell). */
.phabricator-inline-summary-table td.inline-line-number a.num {
- padding-left: 4px;
+ padding: 4px 8px;
display: block;
- font-weight: bold;
- color: {$blue};
+ color: {$bluetext};
}
-.phabricator-inline-summary-table td.inline-line-number a.num:hover {
- background: {$blue};
- border-radius: 3px;
- color: white;
+.phabricator-inline-summary-table td.inline-line-number:hover a.num {
+ background: rgba(150,150,150,.1);
text-decoration: none;
+ color: {$darkbluetext};
}
.phui-timeline-core-content .phabricator-inline-summary:first-child {

File Metadata

Mime Type
text/plain
Expires
Fri, Mar 14, 11:54 AM (3 w, 1 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7656269
Default Alt Text
D11666.id28076.diff (4 KB)

Event Timeline

Phabricator · Built by Phacility