Quote:
Originally Posted by sinucello
Hi,
very nice mod, just bought the pro-version.
I find two things that could/should be improved:
In the hottest.php action file the defined buttons are listed without checking if they are set inactive:
PHP Code:
$templater = vB_Template::create('dbtech_thanks_statistics_statisticbit'); $templater->register('phrase', $vbphrase['dbtech_thanks_hottest_threads'] . ' - ' . $button['title']); $templater->register('userbits', implode('', $threadbits));
IMHO it would be better to list only those buttons that are set active.
Ajax is nice but for those who are not able to use JS there should be a fallback enabling them to use the function. If this is not possible, please consider to insert the whole button via JS so that those without JS are not bothered with a non functional link after each post.
all the best,
Sacha
|
I'll change the hottest.php file for the upcoming bugfix version, thanks
I don't think the JS change is something we'll rush to do since the vast majority of browsers and users have JS enabled (I assume, as you're the first person in my 8 years of coding that have mentioned it) but if you re-post it as a feature suggestion over at our forum, we'll take it into consideration regardless.
For the future, could you post in the Pro support forum on DBTech? Response times there are much more rapid as I get instant email notif when you start a thread
Fillip