[core] fix 'noWrapInfo' CSS class

[vince-fugnitto]

What it does

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.

How to test

• verify that previous uses (extensions) of noWrapInfo are styled correctly.

Review checklist

• as an author, I have thoroughly tested my changes and carefully followed the review guidelines

Reminder for reviewers

• as a reviewer, I agree to behave in accordance with the review guidelines

Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com

[elaihau]

I understand what this change does however I am unable to reproduce what #6592 (comment) shows (on master branch) ... am i missing something?

[lmcbout]

Same a @elaihau I cannot reproduce this issue with and without this PR.
I used the latest commit: 0f16fe8
The File explorer is not wrapping. Am I missing something as well for testing ?

[vince-fugnitto]

@elaihau @lmcbout as described in the issue ( #6592), the problem occurs in a minimal application, for example the theia-cpp-extensions. If you build and test that repo, you will notice the error that is being fixed with the PR.

---

Update

Here is the bug with the explorer for the theia-cpp-extensions example app:

[elaihau]

could you please define "minimal application" @vince-fugnitto ? Thank you !

[vince-fugnitto]

could you please define "minimal application" @vince-fugnitto ? Thank you !

I tested using the theia-cpp-extensions example application, but a minimal application means an app that did not have as dependencies the extensions as part of the changed files. (ex: search-in-workspace)

[elaihau]

With Vincent's help I am able to reproduce the problem.

I tested it with theia-cpp-extensions and the change worked. Also, I don't see the change break the example-browser of theia.