Added add and remove icons to table renderer fixes #1782 by Tarjei400 · Pull Request #2021 · eclipsesource/jsonforms

@Tarjei400

Added add and remove icons for Table renderer

@CLAassistant

CLA assistant check
All committers have signed the CLA.

@Tarjei400

@Tarjei400

@sdirix

Thanks for the contribution ❤️. We will take a look right after the 3.0 release.

@Tarjei400

@sdirix let me know if I can be of any assistance

@coveralls

Coverage Status

Coverage increased (+0.5%) to 84.866% when pulling e773be5 on t3chy0n:master into 79e1c56 on eclipsesource:master.

lucas-koehler

Choose a reason for hiding this comment

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

@Tarjei400 Thanks again for the contribution :) The add and remove buttons already work great for me. I added some suggestions regarding the buttons itself. Could you take a look at them please?

@Tarjei400

lucas-koehler

Choose a reason for hiding this comment

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

Thanks for the updates. I just have two minor comments left.

@Tarjei400

lucas-koehler

Choose a reason for hiding this comment

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

Thanks again for the updates. LGTM now :)