Перейти к публикации

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

10 пользователей проголосовало

  1. 1. Ваше мнение

    • Да, безусловно
      3
    • Нет надобности
      7
    • Если будет - хорошо
      0


WYSIWYG: Вставка изображения с атрибутами высоты и ширины


Рекомендованные сообщения

Вообще, совершенно не ясно, почему этого нет в движке. Ведь совершенно очевидно, что есть новостные сайты, которые используют в публикациях массу изображений и вручную вписывать 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]

Изменено пользователем tobee
Ссылка на сообщение
Поделиться на других сайтах

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

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

Ссылка на сообщение
Поделиться на других сайтах

  12.03.2015 в 12:37, alex32 сказал:

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

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

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

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

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

Изменено пользователем LADYX
Ссылка на сообщение
Поделиться на других сайтах

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

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

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

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

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

Ссылка на сообщение
Поделиться на других сайтах

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

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

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

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

Ссылка на сообщение
Поделиться на других сайтах

Присоединяйтесь к обсуждению

Вы можете опубликовать сообщение сейчас, а зарегистрироваться позже. Если у вас есть аккаунт, войдите в него для написания от своего имени.

Гость
Ответить в тему...

×   Вставлено в виде отформатированного текста.   Вставить в виде обычного текста

  Разрешено не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отобразить как ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставить изображения напрямую. Загрузите или вставьте изображения по ссылке.

×
×
  • Создать...