Terms of use checkboxes come in the context of sometimes-lengthy registration forms. It would be nice if there were a feature to open the linked node in a new window/tab, and/or in a modal window.

If that's not possible, more generally it would be nice if there was a theme template file that could be overridden, and then this kind of thing could be implemented fairly easily.

CommentFileSizeAuthor
#5 2874340-5.patch2.38 KBfinne
#4 2874340-4.patch2.42 KBfinne
#2 2874340-2.patch2.42 KBLendude
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rootwork created an issue. See original summary.

Lendude’s picture

Lendude
Primary language Dutch
Location Amsterdam
CreditAttribution: Lendude as a volunteer and at Dx Experts commented
Status: Active » Needs review
FileSize
2874340-2.patch2.42 KB

Needed this, so here we go.

finne’s picture

finne
Primary language Dutch
Location Amsterdam
CreditAttribution: finne at Dx Experts commented
Status: Needs review » Reviewed & tested by the community

review OK.
Handy addition.

finne’s picture

finne
Primary language Dutch
Location Amsterdam
CreditAttribution: finne at Dx Experts commented
FileSize
2874340-4.patch2.42 KB

Patch did not apply anymore. Rerolled patch against latest 8.x-2.x-dev.

finne’s picture

finne
Primary language Dutch
Location Amsterdam
CreditAttribution: finne at Dx Experts commented
FileSize
2874340-5.patch2.38 KB

And rerolled again for users who are on the 8.x-2.0beta1 release.

Per’s picture

Per CreditAttribution: Per commented

Patch #5 works great with Drupal 8.7.6 and 8.x-2.0beta1
Patch ready to be added to next beta?

ivnish’s picture

ivnish CreditAttribution: ivnish commented
Status: Reviewed & tested by the community » Closed (works as designed)

del

ivnish’s picture

ivnish CreditAttribution: ivnish commented
Status: Closed (works as designed) » Reviewed & tested by the community

Oh, sorry. It is optional

ivnish’s picture

ivnish CreditAttribution: ivnish commented
Status: Reviewed & tested by the community » Fixed

  • ivnish committed 87ccec8 on 8.x-2.x authored by finne 
    Issue #2874340 by finne, Lendude, ivnish, rootwork, Per: Enable terms of...

Status: Fixed » Closed (fixed)

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

jglynn’s picture

jglynn CreditAttribution: jglynn commented

Sure would be useful if there was also an option to open terms in a modal. Seems like it would be a very common use case.