← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1390793 into lp:widelands

 

> 1. I understand seafaring checkbox when loading game - but dont understand it when saving the game. On save this info should be generated automatically and saved into map. 

Automatially generating this tag is a good idea :)

We should do this in a new branch though, this branch is already big
enough. Time for a new bug.


> 2. Of course everybody will figure it out at the end, but still I think the logic&ergonomy is not as polished as could be. But the situation is not getting workse, so OK....

Maybe we should solicit some feedback on the forum again once this
branch has been merged for a while, to see if it is easy or hard to
figure out.


> 7. I was partially heading to something like "save" and "save as" in other programs. BTW, does something like ctrl+S work in editor (without invoking any kind of menu)? I meant to distinguish the small edit of a map and significant changes to a map. Just an idea.... Or maybe there could be a "floppy" icon somewhere which would only save the map without any queries or so. Just another idea :)

CRTL+S would certainly be nice, and have it pop up something only if the
map is still called "No Name". Time for a new bug.


> But OK, at the end there are two outstanding issues : broken sorting of filenames and that "show *names " button. And I noticed, the sorting is broken even also on "Pl." column. And on third column too.

Yes, the sorting is messed up for any table column. Would you like to
have a go at looking at the sorting algorithm I used? It is probably
someting really stupid and obvious once we spot it.

-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1390793/+merge/254558
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1390793 into lp:widelands.


References