opendev/gerrit
Code Issues Proposed changes

Fix white-space issues with gr-linked-text

Browse Source 
Because the white-space: pre-wrap was being set on an element
far up the DOM tree, a lot of extra whitespace was being rendered
as a result.

Change-Id: I4238c90179793e8b0b8196b16b00b2649ad0230f
This commit is contained in:
Andrew Bonventre 2015-12-21 18:56:09 -05:00 committed by Dave Borowitz
parent 37dcf0092e
commit 4c8549e12f
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
polygerrit-ui/app/elements/gr-change-view.html
View File

@ -91,7 +91,6 @@ limitations under the License.
border-bottom: 1px solid #ddd;
font-family: 'Source Code Pro', Menlo, 'Lucida Console', Monaco, monospace;
overflow-x: auto;
white-space: pre-wrap;
}
gr-file-list {
padding: 0 var(--default-horizontal-margin) 10px;
@ -177,7 +176,7 @@ limitations under the License.
</table>
</section>
<section class="summary">
<gr-linked-text content="[[_commitInfo.message]]"></gr-linked-text>
<gr-linked-text pre content="[[_commitInfo.message]]"></gr-linked-text>
</section>
<gr-file-list id="fileList"
change-num="[[_changeNum]]"

4
polygerrit-ui/app/elements/gr-linked-text.html
View File

@ -20,7 +20,7 @@ limitations under the License.
<dom-module id="gr-linked-text">
<template>
<style>
:host([pre]) {
:host([pre]) div {
white-space: pre-wrap;
word-wrap: break-word;
}
@ -57,4 +57,4 @@ limitations under the License.
}
});
</script>
</dom-module>
</dom-module>