[Select] fix: disabled select when fieldset has disabled attr

[ojj1123]

Description

close #2417

• When there is disabled fieldset, Select component does not be disabled.
• This is because Select.Trigger has onPointerDown handler and even if button is disabled, that handler can be triggered.
• So I added the code that check if there is fieldset disabled.

AS-IS

2024-10-10.14.42.50.mov

TO-BE

2024-10-10.14.42.00.mov

[RndUsername]

Will this react to changes? For example if the fieldset is dynamically disabled?

[ojj1123]

Will this react to changes? For example if the fieldset is dynamically disabled?

@RndUsername
Yeah, It works well. This is because disabled state of fieldset is checked whenever triggering onPointerDown handler.
FYI, HTML select tag works as pointerdown event is triggered using mouse. [mdn] <select>

export const Styled = () => {
  const [disabled, setDisabled] = React.useState(false);

  return (
    <div>
      <button type="button" onClick={() => setDisabled((prev) => !prev)}>
        {disabled ? 'enabled' : 'disabled'}
      </button>
      <div>
        <p>Radix `Select` component</p>
        <fieldset disabled={disabled}>
          {POSITIONS.map((position) => (
            <Label key={position}>
              Choose a number:
              <Select.Root defaultValue="two">
                ...
              </Select.Root>
            </Label>
          ))}
        </fieldset>
      </div>
    </div>
  );
};

2024-10-15.22.16.05.mov

[Sparticuz]

Ran into this today, can we get a review and merge? 🙏

[ryanagillie]

Hey all, it's been 4 months can someone take a look at this?

[ojj1123]

I feel like some people want it to be merged. So if need any help, I will be contributing or reviewing:)

(I want to fix conflicts, but there is no permission to write this branch since I was not assigned on this PR)