WebDev

EhPortal => EhPortal Support => Topic started by: Oldiesmann on November 11, 2018, 03:19:47 PM

Title: Shoutbox issues
Post by: Oldiesmann on November 11, 2018, 03:19:47 PM
Working on setting up a test site to show to members of a site I'll be upgrading soon (also to give me a chance to test things a bit more).

I'm running into a couple of issues with the shoutbox though:

Leaving auto-refresh at 0 seems to make it constantly refresh (instead of disabling it)
The shoutbox is too tall despite the settings being the same as they are on the current site (SMF 2.0.15 and SP 2.3.6).

Current site:
[attach name=sp.png type=image/png]324[/attach]

Test site:
[attach name=ep.png type=image/png]326[/attach]

Any suggestions?

Also it would be best to set the column for refresh to something other than a signed tinyint, as that makes the max possible refresh value 127 (and there's no need to make it signed as negative numbers don't make sense in this context). Something like smallint would be better (allowing for things like 5-minute or even one-hour auto refresh times).
Title: Re: Shoutbox issues
Post by: Chen Zhen on November 11, 2018, 05:11:36 PM

The shoutbox uses the same database & most of the same admin settings but has actually been changed quite a bit.
I'll look at 0 seconds to disable the refresh.
The db size is not going to make much difference, I will not bother to change that.
 
Title: Re: Shoutbox issues
Post by: Oldiesmann on November 11, 2018, 05:27:26 PM
Any idea on the weird layout issue?
Title: Re: Shoutbox issues
Post by: Oldiesmann on November 11, 2018, 07:20:52 PM
Did some digging and figured out the issue. The "message_box" class is forcing the main div to be 200px high regardless of what you set the shoutbox height to...

.shout_box .message_box {
   background: #FFFFFF;
   height: 200px;
   overflow:auto;
   border: 1px solid #CCC;
}

Removing the "height: 200px" part makes it behave the way I expect it to.
Title: Re: Shoutbox issues
Post by: Chen Zhen on November 11, 2018, 08:28:13 PM

My apologies, I did not have time to look until now.
Yes that was overlooked and will be fixed with the next version.
Thanks.

Although I am wondering if I should change it to a percentage due to possible mobile usage.
Title: Re: Shoutbox issues
Post by: Oldiesmann on November 11, 2018, 09:19:11 PM
It seems to work fine with no height specified in that class on my phone (Samsung Galaxy S9). Might be good to specify a max-height for mobile devices maybe.
Title: Re: Shoutbox issues
Post by: Skhilled on November 13, 2018, 09:20:36 AM
Hey OldiesMann! How's it going? :)
EhPortal 1.39.6 © 2024, WebDev