Wait Command Change

[Zanderkat]

I've noticed that the 'Wait' command has changed its presentation in 5.9. In 5.8 and earlier, there was one space printed above "Continue..." and one below, but now there are three above "Continue..." and none below. Not only does this push the link right up against the taskbar for longer readings but now there's this big, empty space between it and the text above. I know this isn't a huge deal but it really spoils the visual presentation to the reader.

Is there something I can do to about this in my game, such as overriding the wait command with a custom function?

[Zanderkat]

Here are some image examples.

Old behavior:

New behavior:

[KVonGit]

Hello,

EDIT

Adding this to game.inituserinterface should fix it (corrected to use endWaitDiv):

JS.eval ("$('#endWaitDiv').insertBefore($('#txtCommandDiv'));")

---

OLD RECOMMENDATION

It seems like @Pertex has a fix for this posted somewhere, but I can't find it at the moment. I'll give it a shot, but he might pop in with a better fix. =)

To get rid of that extra space, we can hide the txtCommandDiv element before calling the wait script, and then show the element afterwards.

JS.uiHide ("#txtCommandDiv")
wait {
  JS.uiShow ("#txtCommandDiv")
}

---

Alternatively:

We can modify existing JS functions to keep from having to do it that way every time we call wait.

We could add a JS file to the game with this modified code:

function beginWait() {
    _waitMode = true;
    $("#txtCommand").hide();
    $("#txtCommandDiv").hide(); /* This is new */
    $("#endWaitLink").show();
    markScrollPosition();
}

function sendEndWait() {
    $("#txtCommandDiv").show(); /* This is new */
    UIEvent("EndWait", "");
}

OR... We could add this to our game.inituserinterface script instead of adding a JS file:

      JS.eval ("function beginWait() {    _waitMode = true;    $('#txtCommand').hide();    $('#txtCommandDiv').hide(); /* This is new */    $('#endWaitLink').show();    markScrollPosition();}")
      JS.eval ("function sendEndWait() {    $('#txtCommandDiv').show(); /* This is new */    UIEvent('EndWait', '');}")

[Pertex]

I did not know about this probem but I would use JS to solve it

function div(){
	$div=document.getElementById("endWaitDiv").outerHTML;
	$('#endWaitDiv').remove();
	$('#txtCommandDiv').before($div);
}

@KVonGit I think we can change the order of the divs in the sourcefiles to get the old behaviour

[KVonGit]

I think we can change the order of the divs in the sourcefiles to get the old behaviour

Doh! That's too obvious. If it were a snake, it would have bit me.

// Add this to game.inituserinterface, and that's all that is required
// Corrected to use endWaitDiv
JS.eval ("$('#endWaitDiv').insertBefore($('#txtCommandDiv'));")

[Pertex]

> // Add this to game.inituserinterface, and that's all that is required
> JS.eval ("$('#endWaitLink').insertBefore($('#txtCommandDiv'));")

That is not the best idea. You get this then:

[KVonGit]

Aargh! You are 100% correct. 😁 I put endWaitLink rather than endWaitDiv. Whoopsy! =)

I just edited the posts.

---

🤔 Just to be thorough, how was this bit set up before 5.9?

---

---

We moved that link from inside txtCommandDiv to its own newly created div named endWaitDiv which now comes after txtCommandDiv.

---

5.8

---

5.9

---

After:

	$div=document.getElementById("endWaitDiv").outerHTML;
	$('#endWaitDiv').remove();
	$('#txtCommandDiv').before($div);

---

After JS.eval ("$('#endWaitDiv').insertBefore($('#txtCommandDiv'));"):