Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[core] fix display of toolbar item icons #6514

Merged
merged 1 commit into from
Nov 12, 2019
Merged

[core] fix display of toolbar item icons #6514

merged 1 commit into from
Nov 12, 2019

Conversation

vince-fugnitto
Copy link
Member

What it does

Fixes #6513

  • fixes the display of toolbar item icons when applications are created
    with a subset of available extensions. The common icons are added to core
    for others to use.

The following limited example application was used during testing:

package.json
{
  "private": true,
  "name": "@theia/example-browser",
  "version": "0.12.0",
  "license": "EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0",
  "theia": {
    "frontend": {
      "config": {
        "applicationName": "Theia Browser Example",
        "preferences": {
          "files.enableTrash": false
        }
      }
    }
  },
  "dependencies": {
    "@theia/console": "^0.12.0",
    "@theia/core": "^0.12.0",
    "@theia/debug": "^0.12.0",
    "@theia/editor": "^0.12.0",
    "@theia/filesystem": "^0.12.0",
    "@theia/getting-started": "^0.12.0",
    "@theia/keymaps": "^0.12.0",
    "@theia/languages": "^0.12.0",
    "@theia/monaco": "^0.12.0",
    "@theia/navigator": "^0.12.0",
    "@theia/outline-view": "^0.12.0",
    "@theia/preferences": "^0.12.0",
    "@theia/terminal": "^0.12.0",
    "@theia/workspace": "^0.12.0"
  },
  "scripts": {
    "prepare": "yarn run clean && yarn build",
    "clean": "theia clean && rimraf errorShots",
    "build": "theiaext compile && theia build --mode development",
    "watch": "concurrently -n compile,bundle \"theiaext watch --preserveWatchOutput\" \"theia build --watch --mode development\"",
    "start": "theia start --plugins=local-dir:../../plugins",
    "start:debug": "yarn start --log-level=debug",
    "test": "wdio wdio.conf.js",
    "test-non-headless": "wdio wdio-non-headless.conf.js",
    "coverage:compile": "yarn build --config coverage-webpack.config.js",
    "coverage:remap": "remap-istanbul -i coverage/coverage.json -o coverage/coverage-final.json --exclude 'frontend/index.js' && rimraf coverage/coverage.json",
    "coverage:report:html": "istanbul report --root coverage --format html",
    "coverage:report:lcov": "istanbul report --root coverage --format lcov",
    "coverage": "yarn coverage:compile && yarn test && yarn coverage:remap && yarn coverage:report:lcov && yarn coverage:report:html"
  },
  "devDependencies": {
    "@theia/cli": "^0.12.0"
  }
}

How to test

  • verify that existing toolbar icons are displayed properly

Review checklist

Reminder for reviewers

Signed-off-by: Vincent Fugnitto [email protected]

Fixes #6513

- fixes the display of toolbar item icons when applications are created
with a subset of available extensions. The common icons are added to core
for others to use.

Signed-off-by: Vincent Fugnitto <[email protected]>
@vince-fugnitto vince-fugnitto added the ui/ux issues related to user interface / user experience label Nov 7, 2019
@vince-fugnitto vince-fugnitto self-assigned this Nov 7, 2019
@vince-fugnitto vince-fugnitto added the toolbar issues related to the toolbar label Nov 7, 2019
Copy link
Contributor

@lmcbout lmcbout left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good
Thanks @vince-fugnitto

@vince-fugnitto
Copy link
Member Author

Looks good
Thanks @vince-fugnitto

Thank you for the review! 😃

@vince-fugnitto vince-fugnitto merged commit a341032 into master Nov 12, 2019
@vince-fugnitto vince-fugnitto deleted the vf/GH-6513 branch November 12, 2019 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
toolbar issues related to the toolbar ui/ux issues related to user interface / user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ui/ux] toolbar icons missing in apps with a limited set of dependencies
2 participants