Пришло письмо от Яндекс.Новостей:

Уважаемые партнёры!
Мы планируем изменить формат отображаемых на страницах нашего сервиса логотипов партнёрских сайтов для того, чтобы пользователю легче было найти заголовок предпочитаемого издания на странице сюжета. Фавиконки, которые мы показываем рядом с названием агентства, будут заменены на логотипы.
В этой связи мы немного меняем требования к экспортному файлу.
В <channel> должен быть добавлен элемент yandex:logo, предназначенный для передачи квадратного логотипа. Пример:

<yandex:logo type="square">http://example.net/square_logo.png</yandex:logo>

Атрибут type может быть только square. Рекомендуется для передачи стандартного логотипа использовать тот же элемент yandex:logo, но без атрибута type. Т.е. вместо элемента <image> лучше писать:

<yandex:logo>http://example.net/normal_logo.png</yandex:logo>
<yandex:logo type="square">http://example.net/square_logo.png</yandex:logo>

В связи с этим вопрос - планируется ли доработка модуля?

Заранее спасибо!

CommentFileSizeAuthor
#5 fix_yandex_logo_elements-2425029-5.patch1.32 KBformatC'vt
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

doorskey’s picture

doorskey CreditAttribution: doorskey commented
Issue summary: View changes

  • duozersk committed 6eaf89d on 7.x-1.x 
    Implemented [#2425029] - added yandex:logo channel elements
duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented

doorskey,

Thank you for using the module and for the feature request.

Done, just committed, please test the dev release and report back.

Thanks
AndyB

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented
Status: Active » Fixed
formatC&#039;vt’s picture

formatC'vt CreditAttribution: formatC'vt commented
Status: Fixed » Needs review
FileSize
fix_yandex_logo_elements-2425029-5.patch1.32 KB

Element key name with type=square also should be named yandex:logo and image dimensions must be equal for this element.

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented
Status: Needs review » Fixed

Good-good, thanks for checking.

  • duozersk committed c6e9ea0 on 7.x-1.x authored by formatC'vt 
    Fix for [#2425029] - additional check width = height for type=square and...
prostoproha’s picture

prostoproha CreditAttribution: prostoproha commented

На семерке все прекрасно работает. Планируется ли корректировка кода для версии модуля под Drupal 6?

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented
Version: 7.x-1.1 » 6.x-1.x-dev
Status: Fixed » Patch (to be ported)

  • duozersk committed 6eaf89d on 8.x-1.x 
    Implemented [#2425029] - added yandex:logo channel elements
  • duozersk committed c6e9ea0 on 8.x-1.x authored by formatC'vt
    Fix for [#2425029] - additional check width = height for type=square and...
doorskey’s picture

doorskey CreditAttribution: doorskey commented

Thanks for fast responding! )

Trying to update via drush:

File views_rss_yandex-7.x-1.x-dev.tar.gz_date=1398391728 is corrupt (wrong md5 checksum). [error]
Updating project views_rss_yandex failed. Attempting to roll back to previously installed [error]
version.
Backups were restored successfully.

duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented

Hey!

Drush issues are out of the scope of this module.

AndyB

doorskey’s picture

doorskey CreditAttribution: doorskey commented

Ok! )

  • duozersk committed 0daea27 on 6.x-1.x 
    Implemented [#2425029] - added yandex:logo channel elements
duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented
Status: Patch (to be ported) » Fixed
duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
CreditAttribution: duozersk commented
Issue summary: View changes

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.