Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The visualReport should display images withour a blur #92

Closed
MininAS opened this issue Apr 24, 2021 · 19 comments
Closed

The visualReport should display images withour a blur #92

MininAS opened this issue Apr 24, 2021 · 19 comments

Comments

@MininAS
Copy link
Contributor

MininAS commented Apr 24, 2021

Does it possible to do that visualReport displays images without a blur?
A very small image looks horrible.

Example with the microphom from the google search input field:
image

@jessezach
Copy link
Owner

This is because small icons are saved with larger dimensions, it ends up looking stretched. I can enhance the capture element keyword to additionally take dimensions to save the image in. I will have to check if there's a way to automatically do this

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

Hmm... The visual_images folders contain saved images with own dimensions, i don't see some icons are saved with larger dimensions.
image

When image looking stretched in reporting is good for evaluate and visualisation images by human.
But it is not good when they is blur

@jessezach
Copy link
Owner

My bad. I had fixed the dimensions issue a while back. This issue is because the report html contains fixed width and height for each image in the table. Retaining the original image width and height might probably make the table look uneven. Will check it out. Meanwhile, if you are able to fix the css of the table cell, then you can raise a PR or send me the css changes.

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

fixed width and height for each image in the table - is good!!! Also, I think that's the way it should be

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

I assume that is the browser does blur for small image when it is displayed in bigger size then it is

@jessezach
Copy link
Owner

Yes because the width and height is fixed. Smaller images get stretched width and height wise

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

Found some solution http://jsfiddle.net/namuol/VAXrL/1459/

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

It seems it works
image

@jessezach
Copy link
Owner

Can you raise a PR with the changes? You need to make the changes here https://github.com/jz-jess/RobotEyes/blob/master/RobotEyes/report_generator.py#L315 . From line 315 - 335. I can test and merge it later. You can change the version 1.5.9 in setup.py

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

Do you mean to add style for each ?

@jessezach
Copy link
Owner

jessezach commented Apr 24, 2021

Ah I see. We can't just add it for everything right. This needs to be added only if the image is small? Does it affect regular sized images as well? They should not be pixelated if they're not small

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

Very good question!!! I will check and let known

@MininAS
Copy link
Contributor Author

MininAS commented Apr 24, 2021

I think we can image-rendering: pixelated; for any image. Only perhaps needs to keep geometric proportions?
image

But my previous questiom is means to add style directly to HTML tag? Or we can add it to some common parent tag?

@jessezach
Copy link
Owner

The style needs to be added to the img tags I believe. Not sure if adding to parent will work. You can add a css at the beginning within style tags img { width: 200 image-rendering: pixelated} something like this.

@MininAS
Copy link
Contributor Author

MininAS commented Apr 26, 2021

$ git push origin Issue-92
remote: Permission to jz-jess/RobotEyes.git denied to MininAS.
fatal: unable to access 'https://github.com/jz-jess/RobotEyes.git/': The requested URL returned error: 403

@jessezach
Copy link
Owner

https://opensource.com/article/19/7/create-pull-request-github

@MininAS
Copy link
Contributor Author

MininAS commented Apr 26, 2021

#94

@MininAS
Copy link
Contributor Author

MininAS commented Apr 29, 2021

Should I do somethink else that PR can be pushed?

@jessezach
Copy link
Owner

Nope. I need to test it as soon as I find sometime. Will merge after. You can install your branch into your systems meanwhile to use the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants