Fix Selector Height for Radio/Check Mode

[Satoshi-Sh]

What type of PR is this? (check all applicable)

• Refactor
• Feature
• Bug Fix
• Optimization
• Documentation Update

Description

The height property doesn't work on Radio and Check mode. I updated the Selector component and added styling with sx. I added a unit test for that.

Related Tickets & Documents

• Closes The hight properties in the Selector visual element, in mode='radio' does not work. #2409

How to reproduce the issue

import taipy.gui.builder as tgb
from taipy.gui import Gui

selected_envs = ["env1"]
env_list = [f"env{i}" for i in range(10)]

with tgb.Page() as page:
    tgb.selector(
        value="{selected_envs}",
        lov="{env_list}",
        multiple=True,
        height="150px",
        filter=True,
    )
    tgb.text("Radio mode")
    tgb.selector(
        value="{selected_envs}",
        lov="{env_list}",
        height="150px",
        mode="radio",
    )
    tgb.text("Check mode")
    tgb.selector(
        value="{selected_envs}",
        lov="{env_list}",
        height="150px",
        mode="check",
    )

Gui(page).run()

The height of the selector doesn't work on the Radio and Check mode.

After fix:

Checklist

We encourage you to keep the code coverage percentage at 80% and above.

• This solution meets the acceptance criteria of the related issue.
• The related issue checklist is completed.
• [] This PR adds unit tests for the developed code. If not, why?
• End-to-End tests have been added or updated. If not, why?
• The documentation has been updated, or a dedicated issue created. (If applicable)
• The release notes have been updated? (If applicable)

[FabienLelaquais]

Looks good to me.
For homogeneity sake, I would have used double quotes for all string literals though.
Thanks!

[FredLL-Avaiga]

better return undefined than {}

[FredLL-Avaiga]

well done, tx @Satoshi-Sh

[FabienLelaquais]

👍