Log In
Steel Weather Castform Don't have an account yet? Register now!
.

Forum Thread

Two Palpad bugs [+solve]

Forum-Index Bug Reports Confirmed Bugs Two Palpad bugs [+solve]
SwampFall
OFFLINE
Trainerlevel: 59

Forum Posts: 601
Posted: Thu, 27/09/2018 23:14 (6 Years ago)
Green => add code
red => remove code


Issue #1 : Palpad open in other tab

Only update the chat when has_focus is true:

function updatePrivateChat(){if(!load_lock && has_focus){ ...


Issue #2 : Scrolling down when textbox gets focus / window is resized

This is unnecessary. It's sufficient to scroll down when messages are loaded.

$(document).ready(function() { ... }); (everything inside included)
Riako
OFFLINE
Trainerlevel: 70

Forum Posts: 2,769
Posted: Tue, 12/02/2019 07:00 (5 Years ago)
What bugs are these two code parts actually creating? Are you reporting them because of bad coding or because they are actually causing something unintended that the user can see?
SwampFall
OFFLINE
Trainerlevel: 59

Forum Posts: 601
Posted: Thu, 04/07/2019 22:04 (5 Years ago)
Issue #1: If user X is on tab B and on tab A a palpad chat is open from user Y, user X will never be notified when user Y says something since tab A is "reading" all the messages, even though user X can't see that (until he goes to tab A of course, but it's just annoying)

Issue #2: If you are looking for a message from the past you scroll up a bunch, if then you go to another window (lets say someone sent a message on another site) and come back, palpad will have scrolled down again. This is also somewhat annoying if you're looking for old messages.

Sorry for the late reply, haven't been that active in the forums lately

Edit:
While I'm at it, I should mention you should put a kind of word-wrap for palpad. If someone decides toWriteLikeThisAllTheTime or_maybe_write_like_this_for_some_weird_reason (yes it happens), then it will just leave the palpad window. Just some simple CSS will fix it:

#private_chat div.chat_txt {
position: relative;
margin-left: 30px;
word-wrap: break-word;
}