Attachments not hiding

[Fillip H.]

Can you also bump up my account (Fillip H.) to Admin so I can test out the checkboxes on one of these release posts?

 

[terminator]

Done m8....

 

[Fillip H.]

I believe it's been fixed

 

[terminator]

what was the problem ?

 

[Fillip H.]

Just some incorrect logic in my code.

You'll need to go back and edit other posts to correctly set the flag, but it should work for all future posts

 

[terminator]

Many thanks for you help, and no doubt I will be visiting again to get some more of your mods.

THANK YOU...

 

[terminator]

Just one more question, is it possible to move the "Thank for this post" link to the side of "Reply"

 

[Fillip H.]

Not currently no, but we're most likely going to add moving capabilities to the next version

 

[terminator]

Another question sorry

Is there any way to make "Require Thanks to see attachments" be checked as default ?

Maybe some code that I could alter myself if it is possible.

 

[Fillip H.]

Open up /dbtech/thanks/hooks/newreply_form_start.php and change the following:

			'requirethanks_attach' 	=> '',

to

			'requirethanks_attach' 	=> 'checked="checked"',

 

[hornstar]

I thanked your post in this thread, but I still can't see the attachment. I even refreshed the page, but it still says I need to thank it.

 

[Fillip H.]

The post requires both Thanks and Like for the attachment to be shown

 

[hornstar]

ic. thanks. confirmed working. However it needs improvements...

At first it said only this: This post requires you to press "Thank for this post" before the attachment can be viewed.

So I thought it only had to be thanked. It should have said: This post requires you to press "Thank for this post and Like this post" before the attachment can be viewed.

But anyway I scrolled back up and it was still only saying: This post requires you to press "Thank for this post" before the attachment can be viewed.
Which was confusing as I had just pressed the button. It was only after refreshing the whole page and reading carefully that: This post requires you to press "Thank for this post" before the attachment can be viewed.
And once again I had to refresh the whole page to then see the attachment.

For a first time user it was not real successful. I think if you improve the wording and make it ajaxed or something then it would be great!

 

[Fillip H.]

AJAXed is what it's supposed to be, but sometimes it's a bit slow so it doesn't update properly.

I'll look into changing the wording though

 

[hornstar]

Alright thanks.

I tested it again. I am using FF 3.6.6

I waited about 10 minutes ....But no Ajax was working. I still had to manually refresh the page.

 

[terminator]

It works a treat for me now, thanks for the manual code and auto refresh does work

 

[terminator]

Is there any way to have the options completly hidden from the user now that the attchments box is now checked.

Like any code that will remove it off screen so that the user cannot uncheck the box ?

 

[Fillip H.]

Hiding it, no. But you can do this:

			'requirethanks_attach' 	=> 'checked="checked" disabled="disabled"',

Same file and line as before.