Bug 232031 – [macOS] Update appearance of <datalist> indicator

• Status: RESOLVED
• Resolution: FIXED
• Component: Forms

[Aditya Keerthi]

The appearance of system combo boxes was updated in Big Sur. The <datalist> indicator should have the new button style.

[Aditya Keerthi]

rdar://84474135

[Aditya Keerthi]

Created attachment 441915 [details]
Patch

[Aditya Keerthi]

Created attachment 441923 [details]
Patch

[Wenson Hsieh]

Comment on attachment 441923 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=441923&action=review

> Source/WebCore/rendering/RenderThemeMac.mm:1019
> +        if (auto* buttonElement = input.dataListButtonElement())
> +            coreUIState = (__bridge NSString *)(buttonElement->active() ? kCUIStatePressed : kCUIStateActive);
> +        else
> +            coreUIState = (__bridge NSString *)kCUIStateActive;

Nit - might look cleaner with a top-level `else if` like:

```
if (input.isPresentingAttachedView())
    coreUIState = (__bridge NSString *)kCUIStatePressed;
else if (auto* buttonElement = input.dataListButtonElement())
    coreUIState = (__bridge NSString *)(buttonElement->active() ? kCUIStatePressed : kCUIStateActive);
else
    coreUIState = (__bridge NSString *)kCUIStateActive;
```

[Aditya Keerthi]

Created attachment 442045 [details]
Patch for landing

[Aditya Keerthi]

(In reply to Wenson Hsieh from comment #4)
> Comment on attachment 441923 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=441923&action=review
> 
> > Source/WebCore/rendering/RenderThemeMac.mm:1019
> > +        if (auto* buttonElement = input.dataListButtonElement())
> > +            coreUIState = (__bridge NSString *)(buttonElement->active() ? kCUIStatePressed : kCUIStateActive);
> > +        else
> > +            coreUIState = (__bridge NSString *)kCUIStateActive;
> 
> Nit - might look cleaner with a top-level `else if` like:
> 
> ```
> if (input.isPresentingAttachedView())
>     coreUIState = (__bridge NSString *)kCUIStatePressed;
> else if (auto* buttonElement = input.dataListButtonElement())
>     coreUIState = (__bridge NSString *)(buttonElement->active() ?
> kCUIStatePressed : kCUIStateActive);
> else
>     coreUIState = (__bridge NSString *)kCUIStateActive;
> ```

Done.

Thanks for the review!

[EWS]

Committed r284626 (243347@main): <https://commits.webkit.org/243347@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 442045 [details].