Feature Improvement: Status controls could have a way to have icons instead of the default letters

[gbritoda]

Description For the Status GUI control, instead of showing E, S, I and W, it would be nice to have a way to put icons in it instead.

Acceptance Criteria

• [ ] Ensure new code is unit tested, and check code coverage is at least 90%
• [ ] Propagate any change on the demos and run all of them to ensure there is no breaking change
• [ ] Ensure any change is well documented
• [ ] Status control from TaiPy gui can have Icons instead of the default letters

[pravintargaryen]

Please assign this to me

[jrobinAV]

Hello @pravintargaryen,

Our core maintainer team still needs to review this issue. We are still determining if or how we want to include it in the taipy library. Please give us some time to review the issue before engaging it. This should not be long.

Meanwhile, I invite you to look at the issues with the labels "good first issue" or "Help wanted". These tickets have been reviewed, approved and are opened for a contribution.

thank you.

[jrobinAV]

Hello @pravintargaryen,

The issue has been reviewed. Thank you for volunteering. Go ahead, give it a try. This is not so easy, so make sure you get in touch with the appropriate maintainers: @FredLL-Avaiga @FabienLelaquais @dinhlongviolin1 @namnguyen20999

[FabienLelaquais]

Thank you @jrobinAV and @pravintargaryen

The real difficulty is going to be to keep the control's property API backward compatible. We certainly can help designing this.

[jrobinAV]

Hello @pravintargaryen,

Any news on the issue?

Cheers,

[pravintargaryen]

Hi @jrobinAV,

Apologies for being behind schedule

Working on new icons from Material UI for the frontend status component. Have to test it though.

\taipy\frontend\taipy-gui\src\components\Taipy\Status.tsx \taipy\frontend\taipy-gui\src\components\Taipy\StatusList.tsx

[jrobinAV]

No pb. Take the time you need, of course.

[FredLL-Avaiga]

Hi @jrobinAV,

Apologies for being behind schedule

Working on new icons from Material UI for the frontend status component. Have to test it though.

\taipy\frontend\taipy-gui\src\components\Taipy\Status.tsx \taipy\frontend\taipy-gui\src\components\Taipy\StatusList.tsx

Are the new icons "in line" with the icons we already use ? I thought you'd design a way to provide external icons ie allows the developer to add icons without changing the taipy-gui bundle.

[github-actions[bot]]

This issue has been labelled as "🥶Waiting for contributor" because it has been inactive for more than 14 days. If you would like to continue working on this issue, please add another comment or create a PR that links to this issue. If a PR has already been created which refers to this issue, then you should explicitly mention this issue in the relevant PR. Otherwise, you will be unassigned in 14 days. For more information please refer to the contributing guidelines.

[pravintargaryen]

Approaching PR

[pravintargaryen]

PR https://github.com/Avaiga/taipy/pull/1413

[jrobinAV]

Thank you @pravintargaryen. The maintainers team is reviewing your PR.

[github-actions[bot]]

This issue has been labelled as "🥶Waiting for contributor" because it has been inactive for more than 14 days. If you would like to continue working on this issue, please add another comment or create a PR that links to this issue. If a PR has already been created which refers to this issue, then you should explicitly mention this issue in the relevant PR. Otherwise, you will be unassigned in 14 days. For more information please refer to the contributing guidelines.

[pravintargaryen]

Made a glitch from my end https://github.com/Avaiga/taipy/pull/1413. Will resend the PR.

[pravintargaryen]

Sent a pull request with updated branch details https://github.com/Avaiga/taipy/pull/1580