Merge pull request #79 from cloudbees/feature/UX-199
[feature/UX-199] Empty cells should contain em dash
This commit is contained in:
commit
3f5bd736dd
|
@ -48,8 +48,8 @@ export class Branche extends Component {
|
|||
<td><WeatherIcon score={pipeline.weatherScore} /></td>
|
||||
<td>{latestRun.result}</td>
|
||||
<td>{decodeURIComponent(branch)}</td>
|
||||
<td>{idCommit}</td>
|
||||
<td>{msgCommit}</td>
|
||||
<td>{idCommit || '-'}</td>
|
||||
<td>{msgCommit || '-'}</td>
|
||||
<td>{moment(latestRun.endTime).fromNow()}</td>
|
||||
</tr>);
|
||||
}
|
||||
|
|
|
@ -33,7 +33,7 @@ export default class Pipeline extends Component {
|
|||
| {numberOfFailingPullRequests} failing
|
||||
</td>);
|
||||
} else {
|
||||
multiBranch = multiPr = (<td></td>);
|
||||
multiBranch = multiPr = (<td> - </td>);
|
||||
}
|
||||
|
||||
return (<tr key={name}>
|
||||
|
|
|
@ -15,8 +15,8 @@ export default class PullRequest extends Component {
|
|||
return (<tr key={latestRun.id}>
|
||||
<td>{latestRun.result}</td>
|
||||
<td>{latestRun.id}</td>
|
||||
<td>{pullRequest.title}</td>
|
||||
<td>{pullRequest.author}</td>
|
||||
<td>{pullRequest.title || '-'}</td>
|
||||
<td>{pullRequest.author || '-'}</td>
|
||||
<td>{moment(latestRun.endTime).fromNow()}</td>
|
||||
</tr>);
|
||||
}
|
||||
|
|
|
@ -26,9 +26,9 @@ export default class Runs extends Component {
|
|||
return (<tr key={data.id}>
|
||||
<td>{data.result}</td>
|
||||
<td>{data.id}</td>
|
||||
<td>{changeset && changeset.commitId && changeset.commitId.substring(0, 8)}</td>
|
||||
<td>{changeset && changeset.commitId && changeset.commitId.substring(0, 8) || '-'}</td>
|
||||
<td>{decodeURIComponent(data.pipeline)}</td>
|
||||
<td>{changeset && changeset.comment}</td>
|
||||
<td>{changeset && changeset.comment || '-'}</td>
|
||||
<td>
|
||||
{duration} minutes
|
||||
</td>
|
||||
|
|
|
@ -57,7 +57,7 @@ describe("pipeline component simple rendering", () => {
|
|||
// simple element has no children
|
||||
assert.equal(children[2].type, 'td');
|
||||
assert.isObject(children[2].props);
|
||||
assert.isUndefined(children[2].props.children);
|
||||
assert.equal(children[2].props.children, ' - ');
|
||||
});
|
||||
|
||||
});
|
||||
|
|
Loading…
Reference in New Issue