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.
Comment | File | Size | Author |
---|---|---|---|
#3 | print_title_space.jpg | 54.56 KB | jpereza |
Comment | File | Size | Author |
---|---|---|---|
#3 | print_title_space.jpg | 54.56 KB | jpereza |
Comments
Comment #1
dicreat CreditAttribution: dicreat commentedSorry for mistake in post below with tag
<a>
:(Comment #2
jcnventura CreditAttribution: jcnventura commentedIndeed it does, but why is this a problem?
There is a step to convert all spaces to %20 so that when I try to split on spaces to detect the href attribute I can be sure that it is an attribute and not an href inside an attribute value.
The side effect is that all spaces are no longer spaces. This is no problem in href attributes as a later urldecode call converts them all back into spaces. Other attributes should no longer be relevant (such as the title attribute). So can you tell me why it is important?
Thanks,
João Ventura
Comment #3
jpereza CreditAttribution: jpereza commentedIt is important because the original link had not that %20 in the title attribute, and now it has. Really, it is not a very important bug, but it is a bug.
You could be thinking that the title attribute does not shows when printed in paper, so it is not important. Yes, that's right.
But just figures that you could be using the printer-friendly page in order to view a book page with all of its children pages. In this scenario, you could click in the links and hold the cursor over the link to see the title of that link. Then, you would see the %20 that difficult reading the tooltip text.
See the attached image for an example.
PS: Very good module!
Comment #4
jcnventura CreditAttribution: jcnventura commentedCode to remove the added %20s has now been committed to CVS.
João
Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.