Naked anchors look ugly and are awkward to css.
Let's backport prefix/suffix options from D8.

CommentFileSizeAuthor
#3 add_prefix_suffix_option-2873822-3.patch2.5 KBdaiwik.addweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

axel.rutz created an issue. See original summary.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin at MachbarMacher commented
Title: Add option to wrap » Add prefix/suffix option
Version: 8.x-1.x-dev » 7.x-1.x-dev
Issue summary: View changes
Issue tags: -Needs backport to D7

Ah, in D8 we have prefix and suffix. Not so elegant but works well enuff.
Might be nice to have in 7 too.

daiwik.addweb’s picture

daiwik.addweb CreditAttribution: daiwik.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Active » Needs review
FileSize
add_prefix_suffix_option-2873822-3.patch2.5 KB

Please apply attached patch for the same.

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin at MachbarMacher commented

Kudos. Code looks solid.

jludwig’s picture

jludwig CreditAttribution: jludwig at SciShield commented
Status: Needs review » Reviewed & tested by the community

jludwig’s picture

jludwig CreditAttribution: jludwig as a volunteer commented
Status: Reviewed & tested by the community » Fixed

I've tested this and the code has had more than one pair of eyes. Committed.

Status: Fixed » Closed (fixed)

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