Skip to content

Commit

Permalink
Fix 'noWrapInfo' CSS class
Browse files Browse the repository at this point in the history
Fixes #6592

- fixes the issue where `noWrapInfo` is referenced in `core`
without actually having its CSS defined. The `noWrapInfo` is currently
available from other extensions, and if those extensions are not included
the styling is off.
- removes duplication of CSS properties.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
  • Loading branch information
vince-fugnitto committed Nov 21, 2019
1 parent 2c6df90 commit 2a33414
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 23 deletions.
6 changes: 6 additions & 0 deletions packages/core/src/browser/style/sidepanel.css
Original file line number Diff line number Diff line change
Expand Up @@ -231,3 +231,9 @@
width: 18px;
background-repeat: no-repeat;
}

.noWrapInfo {
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
}
8 changes: 1 addition & 7 deletions packages/extension-manager/src/browser/style/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,6 @@
display: flex;
}

.noWrapInfo {
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
}

.row {
width: 100%;
}
Expand Down Expand Up @@ -92,4 +86,4 @@
}

@import './extension-sidebar.css';
@import './extension-detail.css';
@import './extension-detail.css';
3 changes: 0 additions & 3 deletions packages/git/src/browser/style/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,6 @@
}

.theia-git .noWrapInfo {
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
width: 100%;
}

Expand Down
6 changes: 0 additions & 6 deletions packages/plugin-ext/src/main/browser/style/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,6 @@
display: flex;
}

.noWrapInfo {
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
}

.row {
width: 100%;
}
Expand Down
3 changes: 0 additions & 3 deletions packages/scm/src/browser/style/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,7 @@

.theia-scm .noWrapInfo {
width: 100%;
overflow: hidden;
white-space: nowrap;
align-items: center;
text-overflow: ellipsis;
}

.theia-scm .space-between {
Expand Down
5 changes: 1 addition & 4 deletions packages/search-in-workspace/src/browser/styles/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -263,10 +263,7 @@
background: var(--theia-word-highlight-replace-color0);
}

.noWrapInfo {
white-space: nowrap;
overflow: hidden;
text-overflow: ellipsis;
.t-siw-search-container .noWrapInfo {
width: 100%;
}

Expand Down

0 comments on commit 2a33414

Please sign in to comment.