[Add-on] Welcome Mod 2.0.0 (Beta)

[poppertom69]

Will test it out! looks good though

[Ladysarajane]

Finally able to test out the latest version. I had a problem with subsilver2 overall header.
All the tables underneath the Welcome Mod table were shorter in width. Finally figured it out though.
On the overall header you have:

Spoiler:

<div id="datebar">
<table class="tablebg" align="center" width="100%" cellspacing="1">
<tr>
<td class="row1" width="100%"><span class="gensmall">
<strong>
<div style="float:right;">
<!-- IF S_USER_LOGGED_IN -->{LAST_VISIT_DATE}</strong><!-- ELSE --></strong>{CURRENT_TIME}<!-- ENDIF -->
<!-- IF S_USER_LOGGED_IN --><br />{CURRENT_TIME}<!-- ENDIF --></span></div>
<!-- IF not S_IS_BOT and S_USER_LOGGED_IN -->
<span class="gensmall">
<strong>{S_TIME} {WELCOME_REGISTERED}</strong>
<!-- ELSE -->
<strong>{S_TIME} {WELCOME_GUEST}</strong>
</span>
<!-- ENDIF -->
</div>
</div>
</tr></td></table>

I changed it to this:

Spoiler:

<div id="datebar">
<table class="tablebg" align="center" width="100%" cellspacing="1">
<tr>
<td class="row1" width="100%"><span class="gensmall">
<strong>
<div style="float:right;">
<!-- IF S_USER_LOGGED_IN -->{LAST_VISIT_DATE}</strong><!-- ELSE --></strong>{CURRENT_TIME}<!-- ENDIF -->
<!-- IF S_USER_LOGGED_IN --><br />{CURRENT_TIME}<!-- ENDIF --></span></div>
<!-- IF not S_IS_BOT and S_USER_LOGGED_IN -->
<span class="gensmall">
<strong>{S_TIME} {WELCOME_REGISTERED}</strong>
<!-- ELSE -->
<strong>{S_TIME} {WELCOME_GUEST}</strong>
</span>
<!-- ENDIF -->
</div></td>
</tr></table>
</div>

Now all the tables on the forum are the same size.

[stitch626]

Thanks Jenice, but this was my intent to make the welcome message bar stand out from all the rest . Also I wanted to make sure there wouldn't be any text wrapping going on if someone wanted to have a longer message. That is how it is on my testboard. I guess if people don't like it, they can always change it. Everyone is going to have their own preference how they want their table but there will only be one set of instructions for the templates so I guess before I submit this mod to .com I will have to make up my mind how if I want to change it. I guess I will see if anyone else comments on this.

[dbsmoker]

cool MOD, thx!

i've been playing with and customizing it on my localhost. Will let you know when I put it on my live board if you're interested.

[stitch626]

Thanks dbsmoker,

All feedback counts. If I don't hear of any problems in the next few weeks I am going to resubmit this mod.

[lefty74]

hi stitch,
great job on reworking this mod

Code: Select all

INSERT INTO `phpbb_config` VALUES ('time_afternoon', 'Good Afternoon', '0');
INSERT INTO `phpbb_config` VALUES ('time_early', 'Good morning, you\'re up early', '0');
INSERT INTO `phpbb_config` VALUES ('time_evening', 'Good evening', '0');
INSERT INTO `phpbb_config` VALUES ('time_late', 'Good gosh you\'re up late ', '0');
INSERT INTO `phpbb_config` VALUES ('time_morning', 'Well good morning', '0');

I think this would work better

Code: Select all

INSERT INTO `phpbb_config` (`config_name`, `config_value`) VALUES ('time_afternoon', 'Good Afternoon');
INSERT INTO `phpbb_config` (`config_name`, `config_value`) VALUES ('time_early', 'Good morning, you\'re up early');
INSERT INTO `phpbb_config` (`config_name`, `config_value`) VALUES ('time_evening', 'Good evening');
INSERT INTO `phpbb_config` (`config_name`, `config_value`) VALUES ('time_late', 'Good gosh you\'re up late ');
INSERT INTO `phpbb_config` (`config_name`, `config_value`) VALUES ('time_morning', 'Well good morning');


i hope you do not mind me suggesting something:
have the language variables in from the previous version and if the $config data is empty use the the vars in the language file, this way it can be customised(into one language) but still be used for multi-language boards. something like

Code: Select all

            'TIME_AFTERNOON'                           => (!empty($config['time_afternoon'])) ? $config['time_afternoon'] : $user->lang['TIME_AFTERNOON'],

One of the major changes was: The time of day messages were called upon using a java script code. I was told that is a no no.

I still don't understand why Javascript could not be used here as it now takes the message from a personalised message to a standard guest user message, regardless of what time it is for the visitor (not logged in).
anyway, one suggestion i would have (not sure whether this is MODDB approved) is to use the JS and put a <noscript></noscript> part that uses the php code in case JS is disabled/not supported.

This is not meant to be negative, just some food for thought

Keep up the good work mate

[voice903fm]

I'm new to these downloads. I've download the zip file, however the instruction doc don't tell me what to do from this point. What do I upload to my server?

There a lot of folders and files but no installation instructions.

[Ladysarajane]

The installation instructions is the XML document. Open that in your browser and it will explain how to install the mod.

[sbdanielb]

I like this mod but i would like to change the colour the block that it shows in, i have seen that the block it shows in is the same as the navbar.

How would i change the colour of that block but not change the colour of the navbar?

[sbdanielb]

This happened out of the blue, it only shows like this for guests, why?

Can see it in screenshot below.