Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
file aggregator-item.html.twig:
<article {{ attributes }}>
<h2><a href="{{ feed_url }}">{{ feed_title }}</a></h2>
{% if content %}
{{ content }}
{% endif %}
<footer>
{% if source_url %}
<a href="{{ source_url }}" class="feed-item-source">{{ source_title }}</a> -
{% endif %}
<time>{{ source_date }}</time>
{% if categories %}
<div>
{{ 'Categories'|t }}: {{ categories|join(', ') }}
</div>
{% endif %}
</footer>
</article>
the aggregator do build its own wrapper markup thats not effected by twig in function _aggregator_page_list which isnt ideal & makes it confusing that we cant change that part in the twig files.
$build = array(
'#type' => 'container',
'#attributes' => array('class' => array('aggregator-wrapper')),
);
Comment | File | Size | Author |
---|---|---|---|
#1 | 2003816-aggregator.diff | 2.16 KB | mortendk |
Comments
Comment #1
mortendk CreditAttribution: mortendk commentedRemoved the
.feed-item
aggregator-item.html.twig is now html5
the categories have easier syntax for later costimyzation
Comment #2
mortendk CreditAttribution: mortendk commentedi get a little bit sad by seeing this in the aggregator.module:
should i open up a seperate issue for this or is it wiser we hammer it all out into one pathc ?
Comment #3
hrmoller CreditAttribution: hrmoller commentedI think the
container
issue is on higher level than the aggregator.module itself asis used widely across core.
Regarding #2 do you think it would be sufficient to use the
l()
for creating the link?Comment #4
twistor CreditAttribution: twistor commentedThis will have a trailing comma if there's only one category.
Comment #5
mortendk CreditAttribution: mortendk commentedComment #6
mortendk CreditAttribution: mortendk commentedComment #7
mortendk CreditAttribution: mortendk commentedComment #8
ParisLiakos CreditAttribution: ParisLiakos commented@ #2:
#1763964: Use #type => link for theme_aggregator_block_item()
this should removed and be a #type => link i think
Comment #19
SpokjeThe
aggregator
module has been removed from Core in10.0.x-dev
and now lives on as a contrib module.Issues in the Core queue about the
aggregator
module, like this one, have been moved to the contrib module queue.Comment #20
larowlan_aggregator_page_list is long gone