Author |
Topic |
bobby131313
Senior Member
USA
1163 Posts |
Posted - 29 May 2008 : 13:50:44
|
You need to assign the id to the <a> tag and you don't need the </id> tag. < |
Switch the order of your title tags |
Edited by - bobby131313 on 29 May 2008 13:52:01 |
|
|
bobby131313
Senior Member
USA
1163 Posts |
Posted - 29 May 2008 : 14:26:15
|
Also need to change...
".keywords a {color:yellow;font-weight:bold;background-color:yellow;)" & vbNewLine & _ to...
"#keywords {color:yellow;font-weight:bold;background-color:yellow;}" & vbNewLine & _
< |
Switch the order of your title tags |
Edited by - bobby131313 on 29 May 2008 14:26:41 |
|
|
cripto9t
Average Member
USA
881 Posts |
Posted - 29 May 2008 : 14:31:08
|
I think I got it right now, thanks. Are your topics still being blocked?
< |
_-/Cripto9t\-_ |
|
|
cripto9t
Average Member
USA
881 Posts |
Posted - 29 May 2008 : 14:34:02
|
quote: Originally posted by bobby131313
Also need to change...
".keywords a {color:yellow;font-weight:bold;background-color:yellow;)" & vbNewLine & _ to...
"#keywords {color:yellow;font-weight:bold;background-color:yellow;}" & vbNewLine & _
Aaah yea that would help to
Done< |
_-/Cripto9t\-_ |
|
|
bobby131313
Senior Member
USA
1163 Posts |
|
cripto9t
Average Member
USA
881 Posts |
Posted - 29 May 2008 : 16:53:01
|
Yea, I only tested default, forum and topic and it blocked default and forum but checks topic. I know your forum is pretty modded up, do you still have this line of code in "inc_header" (near the top)? strScriptName = request.servervariables("script_name") That's the variable I'm using to check the page.< |
_-/Cripto9t\-_ |
|
|
bobby131313
Senior Member
USA
1163 Posts |
|
bobby131313
Senior Member
USA
1163 Posts |
Posted - 29 May 2008 : 17:57:07
|
Just double checked everything and it looks good, but still unlinked on topic.asp. I'll keep pokin' around.< |
Switch the order of your title tags |
|
|
cripto9t
Average Member
USA
881 Posts |
Posted - 30 May 2008 : 08:08:43
|
This should fix the "word within word" and "link within link" bugs.
In the ChkKey() function replace this linefString = Replace(fString,keywords(i),keyreplace(i), 1, -1, 1) with this fString = Replace(fString, " " & keywords(i) & " ", " " & keyreplace(i) & " ", 1, -1, 1) < |
_-/Cripto9t\-_ |
|
|
leatherlips
Senior Member
USA
1838 Posts |
|
leatherlips
Senior Member
USA
1838 Posts |
|
cripto9t
Average Member
USA
881 Posts |
Posted - 30 May 2008 : 17:50:16
|
The original code has been updated with all the bug fixes mentioned so far.
Give me a little time, I think I have a fool proof way or restricting it to certian files.
Also, I just thought of a few things that could go wrong with the "word within word" and "link within link" fix I posted (but its still better than it was). So give me some time and I'll see if can come up with something better.< |
_-/Cripto9t\-_ |
|
|
leatherlips
Senior Member
USA
1838 Posts |
|
bobby131313
Senior Member
USA
1163 Posts |
Posted - 31 May 2008 : 12:30:50
|
One other little thing that's not that important really.
Is there any way to preserve the originally posted case? It converts everything to lower case.
Thanks for your work, it is appreciated. < |
Switch the order of your title tags |
|
|
bobby131313
Senior Member
USA
1163 Posts |
Posted - 01 June 2008 : 19:17:18
|
Another odd little quirk. It seems that it recognizes spaces just fine in the middle of a link phrase, but will not recognize leading or trailing spaces which would be very helpful in resolving some conflicts.< |
Switch the order of your title tags |
|
|
Topic |
|