Необходимо ли это внедрить в движок по умолчанию? 10 пользователей проголосовало

[tobee 16]

Вообще, совершенно не ясно, почему этого нет в движке. Ведь совершенно очевидно, что есть новостные сайты, которые используют в публикациях массу изображений и вручную вписывать width и height каждой из них - трата массы времени...

- Обнаружено, что DLE не вставляет атрибуты ширины (width) и высоты (height) в изображения после загрузки при использовании WYSIWYG редакторов, теперь атрибуты будут вставляться нормально, как в массовой вставке изображений так и в одиночной.

Исправим сие невероятное упущение разработчика:

*** ВСЕ ДЕЛАТЬ В ФАЙЛЕ: /engine/ajax/upload.php

найти ВСЕ:

data-type=\"image\"

заменить на:

data-type=\"image\" data-width=\"{$img_info[0]}\" data-height=\"{$img_info[1]}\"[/code]




найти:

[code]links[t] = buildimage ($(elmnt).data('src'), true);[/code]

заменить на:

[code]links[t] = buildimage ($(elmnt).data('src'), true, $(elmnt).data('width'), $(elmnt).data('height'));[/code]

найти:

[code]function buildimage( image, mass ) {[/code]

заменить на:

[code]function buildimage( image, mass, width, height ) { var image_size = ''; if (width>0) image_size += ' width="'+width+'" '; if (height>0) image_size += ' height="'+height+'" ';[/code]

найти:

[code]content = '<img src="'+ image +'" />';[/code]

заменить на:

[CODE]content = '<img src="'+ image +'" '+image_size+' />';[/code]

найти:

[code]content = '<img src="'+ image +'" style="float:' + align+ ';" />';[/code]

заменить на:

[code]content = '<img src="'+ image +'" style="float:' + align+ ';" '+image_size+' />';[/code]

найти:

[code]content = '<img src="'+ image +'" alt="'+ imagealt +'" title="'+ imagealt +'" />';[/code]

заменить на:

[code]content = '<img src="'+ image +'" alt="'+ imagealt +'" title="'+ imagealt +'" '+image_size+' />';[/code]

найти:

[code]content = '<img src="'+ image +'" alt="'+ imagealt +'" title="'+ imagealt +'" style="float:' + align+ ';" />';[/code]

заменить на:

[code]content = '<img src="'+ image +'" alt="'+ imagealt +'" title="'+ imagealt +'" style="float:' + align+ ';" '+image_size+' />';[/code]

найти:

[code]content = buildimage (url, false);[/code]

заменить на:

[code]var width = $('.box').find('.uploadfile[data-src="'+url+'"]').data('width'); var height = $('.box').find('.uploadfile[data-src="'+url+'"]').data('height'); content = buildimage (url, false, width, height);[/code]

Изменено 12 марта 2015 пользователем tobee

[odys 384]

tobee, вообще считаю бредовой затеей!

А если дизайн адаптивный? А если у меня блок с полной новостью меньше по ширине, чем картинка? А если я забочусь о валидности?

Короче бред...

[alex32 934]

Да как то атрибуты width как и height в теге img хоть и имеют место быть, но ведь некошерно, для этого css есть

[tobee 16]

Да как то атрибуты width как и height в теге img хоть и имеют место быть, но ведь некошерно, для этого css есть

Не понятно, зачем в этом случае еще и CSS-то? Не подскажете?

[celsoft 5 996]

- Обнаружено, что DLE не вставляет атрибуты ширины (width) и высоты (height) в изображения после загрузки при использовании WYSIWYG редакторов, теперь атрибуты будут вставляться нормально, как в массовой вставке изображений так и в одиночной.

data-type="image" data-width="{$img_info[0]}" data-height="{$img_info[1]}"

Вы дублируете одно и тоже. При загрузке картинки уже создались с этим размером, т.е. по факту имеют указанный размер, зачем еще раз их вписывать в сам HTML тег? Это получается что то вроде масло масленное.

А что вставлять, уменьшенную копию, оригинал, или средную копию вы при вставке выбираете, равно как и то например увеличивать картинку при этом или нет.

[alex32 934]

Не понятно, зачем в этом случае еще и CSS-то? Не подскажете?

не ЕЩЕ а ВМЕСТО.

[Dj Dance 185]

При нынешних тенденциях, затея с размерами абсолютно плохая.

[LADYX 90]

Исправим сие невероятное упущение разработчика:

Что называется полный ппц!!! Прочитал, и тут же у меня это вызвало бурную реакцию. Уважаемые разработчики! И не подумайте даже читать эту тему, и тем более пытаться воплощать ее в реальность. Это какое-то инопланетное вмешательство, которое пытается вернуть нас в ужасное прошлое. Как вспомнишь, так вздрогнешь. А тут такое, да еще и с опросом, ужас! И слова такие умные подобраны, караул!

<b>tobee</b>, да вы какой-то самомазохист что ли, не надоело ли ручками работать? Ну найдите им применение где-нибудь в другом месте, пожалуйста. При всем к вам уважении.

Изменено 12 марта 2015 пользователем LADYX

[tobee 16]

Вы дублируете одно и тоже. При загрузке картинки уже создались с этим размером, т.е. по факту имеют указанный размер, зачем еще раз их вписывать в сам HTML тег? Это получается что то вроде масло масленое.

Про "масло" не понятно, что дублируется-то?

[celsoft 5 996]

Про "масло" не понятно, что дублируется-то?

Вы картинкам назначаете тот же размер, что им был назначен при их загрузке, и являются для них фактическими на сервере, т.е. вы вторично указываете и так фактические размеры картинки. Говоря простым языком это выглядит так, браузер видит что картинка имеет например ширину в 500 пикселей, а вы ему еще раз пишите, да да это именно 500 пикселей. Т.е. получается выражение "масло маслянное", а это лишний код и лишний расход памяти, если вы и хотите указывать размеры, то как минимум ставьте отличные от фактических, иначе это бессмысленно.

[tobee 16]

а это лишний код и лишний расход памяти, если вы и хотите указывать размеры, то как минимум ставьте отличные от фактических, иначе это бессмысленно.

Иными словами, вы говорите мне, что совершенно не нужно указывать ширину и высоту картинки?

Т.е. достаточно указывать так:

<img src="http//" alt />[/CODE]

Вы уверены, что поисковики нормально сканируют картинки сайта, если для них не указаны размеры?

[celsoft 5 996]

Иными словами, вы говорите мне, что совершенно не нужно указывать ширину и высоту картинки?

Абсолютно уверен.

Вы уверены, что поисковики нормально сканируют картинки сайта, если для них не указаны размеры?

Прекрасно сканируют и индексируют