Hello,

I think the ability to allow HTML in the prev, next and home links will be a plus for this module. One may need some HTML to place an image tag instead of plain text. What do you think ?

I will attach a patch to test this.

CommentFileSizeAuthor
#1 entitypager-allow-html-in-next_prev_home-links-2543278-2.patch2.91 KBmihai7221
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mihai7221’s picture

mihai7221 CreditAttribution: mihai7221 commented
FileSize
entitypager-allow-html-in-next_prev_home-links-2543278-2.patch2.91 KB
JmOkay’s picture

JmOkay CreditAttribution: JmOkay as a volunteer commented

Hi Canutza

Thanks for the patch / idea. Yeah good idea, it is also my intention to run the Next and Prev through the de-token function (method). I've applied your patch locally and had a look at it - so thanks. I will use it when adding the de-token functionality on the Next and Prev. De-tokening Next and Prev boxes will also require an UI change to add boxes for URLs (i.e. like the All box functionality, a label box and also a new URL box will be required). With this in mind, I didn't want to complicate the UI / Project whilst in sandbox.

At this stage with this project still being a Sandbox project under review, I did not want to add further functionality until it has moved to being a fully fledged module. That way people don't have to re-review a moving target (module).

So again thanks Canutza, much appreciated.

JmOkay

JmOkay’s picture

JmOkay CreditAttribution: JmOkay as a volunteer and commented
Version: » 7.x-1.1
Status: Active » Fixed

Hi Canutza

I've added your suggestion. I would have added it earlier, but had to wait the best part of a year for the module to be upgraded to full status by Drupal.org, so all good now. Hope you enjoy using the module, it seems to work really well.

Thanks

JmOkay

JmOkay’s picture

JmOkay CreditAttribution: JmOkay as a volunteer and commented
Status: Fixed » Closed (fixed)