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

Superflous html code when using mediarepositoryImageThumb #136

Closed
rgasch opened this issue Nov 8, 2011 · 1 comment
Closed

Superflous html code when using mediarepositoryImageThumb #136

rgasch opened this issue Nov 8, 2011 · 1 comment
Assignees
Milestone

Comments

@rgasch
Copy link

rgasch commented Nov 8, 2011

When you display an image and specify a size using mediarepositoryImageThumb the currently generated code also includes the html tags "width" and "height". From my tests I conclude that these are unncessary as the mediarepositoryImageThumb size designation already ensures that the images has the specified size.

@Guite
Copy link
Owner

Guite commented Nov 8, 2011

There is a good good reason for width and height tags: Performance and Accessibility.

If the html code explicitely includes image dimensions the user agent / browser knows these earlier and can therefore preassign the required space. This can lead to significant improvements from user perspective during the page display as the browser does not change or resize things after another image has been loaded.

@Guite Guite closed this as completed Nov 8, 2011
@Guite Guite self-assigned this Dec 15, 2016
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