RFR: 8302173: Button border overlaps with button icon on macOS system LaF [v2]
Damon Nguyen
dnguyen at openjdk.org
Tue Feb 14 21:14:46 UTC 2023
On Tue, 14 Feb 2023 20:03:20 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:
>> Nevermind, I see your point now that I've reread this. Checking for an icon achieves the same effect without having to check for an img pattern as long as one is adjusted. So, I think the icon check is more robust and cleaner. Thanks!
>
> @DamonGuy Additionally, you might need to make sure the conditional and the fix works under different combinations especially when you have an html with `<img>` tag + icon, both added to the button.
Yes, I tested with these combinations:
Icon with HTML image
<img width="274" alt="Screen Shot 2023-02-14 at 12 30 23 PM" src="https://user-images.githubusercontent.com/96267980/218855067-68186106-77df-4c2b-81d7-2cd24e28f4bb.png">
Icon with HTML text
<img width="270" alt="Screen Shot 2023-02-14 at 1 10 28 PM" src="https://user-images.githubusercontent.com/96267980/218863780-de2293ea-9da6-4db9-9201-4788b7567478.png">
And the `HtmlButtonImageTest` still passes as expected
-------------
PR: https://git.openjdk.org/jdk/pull/12520
More information about the client-libs-dev
mailing list