On Mon, 18 Mar 2024 13:27:56 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>>> > > > I am not sure but to me the images dont seem to match in 
>>> > > > windows11..it seems JEditorPane image is 1 pixel thicker than golden 
>>> > > > images...in which case, it needs to be problemlisted Also, I guess 
>>> > > > this has potential of making it automated by loading the golden 
>>> > > > images and JEditorPane in BufferedImage and then compare...
>>> > > > ![image](https://private-user-images.githubusercontent.com/43534309/313576442-c9b84f20-5b0c-44f2-a9a8-c5568d1b0ef4.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTA3NTM2MTEsIm5iZiI6MTcxMDc1MzMxMSwicGF0aCI6Ii80MzUzNDMwOS8zMTM1NzY0NDItYzliODRmMjAtNWIwYy00NGYyLWE5YTgtYzU1NjhkMWIwZWY0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzE4VDA5MTUxMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWY2NjJmNzRmMzBjMzUyMzUxOWRiODBmZGJmNGU5Zjc4NThmMTUwZjc4NWJkN2NhYWI0ZmE4YWJkYmZkYTg3YjgmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.c3EEHxT_VGBlWLLQkH4xIdbcXBBuvpYoNW3KQfZNgAw)
>>> > > 
>>> > > 
>>> > > You mean to say the border of JEditorPane is 1 pixel thicker ?
>>> > 
>>> > 
>>> > it doesn't look exactly same subjectively...
>>> > ![image](https://private-user-images.githubusercontent.com/43534309/313618601-15dcdcab-fdf4-4bf4-8ef8-7c0689d6b538.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTA3NTQyNDIsIm5iZiI6MTcxMDc1Mzk0MiwicGF0aCI6Ii80MzUzNDMwOS8zMTM2MTg2MDEtMTVkY2RjYWItZmRmNC00YmY0LThlZjgtN2MwNjg5ZDZiNTM4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzE4VDA5MjU0MlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTU3NmZmMGZhNDUyYzM2Y2UyNGExN2QyZjFiNjljZDljMTE2NDdiMmM4NzIwYzgzOTdjYWJmZGY5ZDdjOTkwMTYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.d_TI9VegwGDfOcO0zHhgHhxtOHRiTZudNBS0IXIIGhI)
>>> 
>>> Yeah, but in instructions they say it should look similar, so wasn't sure 
>>> whether to problem list or proceed with conversion. You think we should 
>>> problem list?
>> 
>> I am not sure...I think there was some fix regarding border stroke sometime 
>> back either by @honkar-jdk or @DamonGuy . Maybe expectation should be 
>> somewhat different now in which case...
>
>> > > > > I am not sure but to me the images dont seem to match in 
>> > > > > windows11..it seems JEditorPane image is 1 pixel thicker than golden 
>> > > > > images...in which case, it needs to be problemlisted Also, I guess 
>> > > > > this has potential of making it automated by loading the golden 
>> > > > > images and JEditorPane in BufferedImage and then compare...
>> > > > > ![image](https://private-user-images.githubusercontent.com/43534309/313576442-c9b84f20-5b0c-44f2-a9a8-c5568d1b0ef4.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTA3NTM2MTEsIm5iZiI6MTcxMDc1MzMxMSwicGF0aCI6Ii80MzUzNDMwOS8zMTM1NzY0NDItYzliODRmMjAtNWIwYy00NGYyLWE5YTgtYzU1NjhkMWIwZWY0LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzE4VDA5MTUxMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWY2NjJmNzRmMzBjMzUyMzUxOWRiODBmZGJmNGU5Zjc4NThmMTUwZjc4NWJkN2NhYWI0ZmE4YWJkYmZkYTg3YjgmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.c3EEHxT_VGBlWLLQkH4xIdbcXBBuvpYoNW3KQfZNgAw)
>> > > > 
>> > > > 
>> > > > You mean to say the border of JEditorPane is 1 pixel thicker ?
>> > > 
>> > > 
>> > > it doesn't look exactly same subjectively...
>> > > ![image](https://private-user-images.githubusercontent.com/43534309/313618601-15dcdcab-fdf4-4bf4-8ef8-7c0689d6b538.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTA3NTQyNDIsIm5iZiI6MTcxMDc1Mzk0MiwicGF0aCI6Ii80MzUzNDMwOS8zMTM2MTg2MDEtMTVkY2RjYWItZmRmNC00YmY0LThlZjgtN2MwNjg5ZDZiNTM4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzE4VDA5MjU0MlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTU3NmZmMGZhNDUyYzM2Y2UyNGExN2QyZjFiNjljZDljMTE2NDdiMmM4NzIwYzgzOTdjYWJmZGY5ZDdjOTkwMTYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.d_TI9VegwGDfOcO0zHhgHhxtOHRiTZudNBS0IXIIGhI)
>> > 
>> > 
>> > Yeah, but in instructions they say it should look similar, so wasn't sure 
>> > whether to problem list or proceed with conversion. You think we should 
>> > problem list?
>> 
>> I am not sure...I think there was some fix regarding border stroke sometime 
>> back either by @honkar-jdk or @DamonGuy . Maybe expectation should be 
>> somewhat different n...

> @prsadhuk With scaling 1.0 it'll appear this way. I think it's supposed to be 
> 1.0 scaling and its looking similar now. 
> ![image](https://private-user-images.githubusercontent.com/94159358/313696311-92e4358b-651d-43d9-bdc8-cb4a44aa021c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTA3NzA5MjMsIm5iZiI6MTcxMDc3MDYyMywicGF0aCI6Ii85NDE1OTM1OC8zMTM2OTYzMTEtOTJlNDM1OGItNjUxZC00M2Q5LWJkYzgtY2I0YTQ0YWEwMjFjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDAzMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwMzE4VDE0MDM0M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTc0NTcyZGE3NTBlOWFhY2YxYzk3NWNjZjc4ZDliNmJlZDE5ZWZhMDA3NTc2ZDliYjJmOGJmOTEwOTdlY2M1YWEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.sSwTFVLibyG4g0u0xl92qA8fS5zU08-CP_6UICqH6Ds)

I am not sure..it still seems 1.25 bigger than golden image..."similar" word 
has different connotation and is very subjective....is it intentional that 
"same" was not mentioned?
What if we say, Earth and Mars are similar as they are planets...

-------------

PR Comment: https://git.openjdk.org/jdk/pull/18260#issuecomment-2004016674

Reply via email to