Skip to content

[FEATURE] make news detail page navigation display next and prev news…#390

Merged
dmh merged 1 commit intot3kit:masterfrom
dkd:news-detail-pagination
May 10, 2018
Merged

[FEATURE] make news detail page navigation display next and prev news…#390
dmh merged 1 commit intot3kit:masterfrom
dkd:news-detail-pagination

Conversation

@aennafae
Copy link
Contributor

@aennafae aennafae commented Feb 6, 2018

… item title & date

@aennafae aennafae force-pushed the news-detail-pagination branch from 7f480a1 to 3c11f82 Compare February 7, 2018 08:41
@aennafae aennafae force-pushed the news-detail-pagination branch from 3c11f82 to ef418c5 Compare February 7, 2018 16:20
</f:link.page>
</div>
</f:if>
</f:then>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this tag </f:then> is not needed - cause errors.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, could you explain how to enable navigation for next and prev news in detail view template? I would like to test it, but can't find this feature :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean "plugin.tx_news.settings.detail.showPrevNext = 1" in EXT:theme_t3kit/Resources/Private/Extensions/News/TypoScript/setup.ts? It should be enabled.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you still have problems to test it ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I still can't enable those links, looks like I am missing some config ... :(
But I decided to merge it without testing, hope that everything should be ok. Thanks for the help!

</f:link.page>
</div>
</f:if>
</f:then>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I still can't enable those links, looks like I am missing some config ... :(
But I decided to merge it without testing, hope that everything should be ok. Thanks for the help!

@dmh dmh merged commit c6c3b24 into t3kit:master May 10, 2018
@websi websi deleted the news-detail-pagination branch June 29, 2018 14:30
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

Successfully merging this pull request may close these issues.

3 participants