[wesnoth-commits] [wesnoth/wesnoth] 264f90: Default [print]duration= to 5 seconds, tutorial us...

[Steve Cotton]

  Branch: refs/heads/master
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 264f90eed47ae7574efb739a875bea3cecc4234c
      https://github.com/wesnoth/wesnoth/commit/264f90eed47ae7574efb739a875bea3cecc4234c
  Author: Steve Cotton <steve at octalot.co.uk>
  Date:   2021-06-19 (Sat, 19 Jun 2021)

  Changed paths:
    M data/campaigns/tutorial/utils/utils.cfg
    M data/schema/core/actionwml.cfg
    M src/scripting/game_lua_kernel.cpp

  Log Message:
  -----------
  Default [print]duration= to 5 seconds, tutorial uses "unlimited"

In 1.14, the default was 50 frames, or around 1.7 seconds. In 1.15.4, commit
a9d9e48c72e70fd41ae9759e3894752eb9708596 changed the interpretation of that
number to milliseconds, but missed that this affected the [print] tag; this
left the default time that the text is shown as an unreadable 50ms.

All places in mainline that use [print] specify a duration, so the default
isn't used. Here I've plucked the new value from UtBS S09, where it was chosen
in f405b916a1141c2a4bc3bda3f88bbe0a8b8fe91b.

The special value "unlimited" is now recognised as meaning to display the text
until it's removed by another [print] tag. The tutorial uses this special case
to display the text until the player does the requested move - originally it
displayed the text for 10000 frames (around 40 minutes), which still seemed
reasonable when it changed to 10000ms.