JBIDE-28449: Extensions from platform should have an indicator

[jeffmaury]

Signed-off-by: Jeff MAURY jmaury@redhat.com

Pull Request Checklist

General

• Is this a blocking issue or new feature? If yes, QE needs to +1 this PR

Code

• Are method-/class-/variable-names meaningful?
• Are methods concise, not too long?
• Are catch blocks catching precise Exceptions only (no catch all)?

Testing

• Are there unit-tests?
• Are there integration tests (or at least a jira to tackle these)?
• Is the non-happy path working, too?
• Are other parts that use the same component still working fine?

Function

• Does it work?

[sbouchet]

also, is there any example/extension without the icon yet to test ?

[jeffmaury]

also, is there any example/extension without the icon yet to test ?

Quinoa

[jeffmaury]

LGTM, however the icon is still there ?

It should be a different icon. Previously the icon identified extensions that add sample code, now the icon identifies extensions that are part of the platform

[sbouchet]

LGTM, however the icon is still there ?

It should be a different icon. Previously the icon identified extensions that add sample code, now the icon identifies extensions that are part of the platform

yes, i see the new icon in the UI, my remark is more that the old one is still in the source code repo and should be deleted since it is not used anymore :)

[jeffmaury]

LGTM, however the icon is still there ?

It should be a different icon. Previously the icon identified extensions that add sample code, now the icon identifies extensions that are part of the platform

yes, i see the new icon in the UI, my remark is more that the old one is still in the source code repo and should be deleted since it is not used anymore :)

Sorry I pushed my commit to another PR. Fixed now