URL:
  <http://gna.org/bugs/?22382>

                 Summary: Restore whitespace-aligned columns in map format
                 Project: Battle for Wesnoth
            Submitted by: zookeeper
            Submitted on: Tue 22 Jul 2014 02:00:09 PM UTC
                Category: Feature Request
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group:  None of the others
                  Status: None
                 Privacy: Public
             Assigned to: alarantalara
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
                 Release: 1.11
        Operating System: all

    _______________________________________________________

Details:

In ea3028 (see also 7f5f62 and possibly also 1a27a8), the old method of
aligning up the columns in map data with spaces was dropped. This makes
reading and editing of plain map data a lot harder for no apparent tangible
benefit.

The old way padded each entry to a length of 12 (excluding the preceding
space):


Gs^Fms      , Gs^Fp       , Gg          , Gg
Wo          , Wo          , Ur          , Cud


The new format only includes the preceding space:


Gs^Fms, Gs^Fp, Gg, Gg
Wo, Wo, Ur, Cud


So, I'd suggest restoring the old padding behavior, although the number of
spaces could be reduced if necessary; since a terrain string cannot be longer
than 9 characters, the column width could be reduced from 12 to 9. As
future-proofing, the code should probably handle even longer strings
gracefully, or to signal an error if one is encountered.

P.S. Alarantalara made the change, so I'm assigning to him initially just so
he gets notified, in case he has input.




    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?22382>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Wesnoth-bugs mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-bugs

Reply via email to