kaputtnik has proposed merging
lp:~franku/widelands-website/added-css-link-classes into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/added-css-link-classes/+merge/245346
This branch
kaputtnik has proposed merging
lp:~franku/widelands-website/smiley-and-codeblock-enhancements into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/smiley-and-codeblock-enhancements
Tanks for approve.
> Your changes look fine. Just one question and an answer to your question in
> the code below.
Yes the "#b"'s (appears twice) are unintendet, better: i forgot to remove them.
I used them to trigger an error to debug the code. I haven't found a way to
debug the code (make hal
Hoping i've done it the right way...
Thank you for review and pointing on some mistakes.
And for the hints to grep.
--
https://code.launchpad.net/~franku/widelands-website/smiley-and-codeblock-enhancements/+merge/245742
Your team Widelands Developers is subscribed to branch lp:widelands-website.
:-)
--
https://code.launchpad.net/~franku/widelands-website/smiley-and-codeblock-enhancements/+merge/245742
Your team Widelands Developers is subscribed to branch lp:widelands-website.
___
Mailing list: https://launchpad.net/~widelands-dev
Post to :
kaputtnik has proposed merging lp:~franku/widelands-website/handle_big_images
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/handle_big_images/+merge/247235
This branch has
If i add more than one normal road to each tribe, could i use this branch again
for upload? Or must i create a new branch from trunk?
--
https://code.launchpad.net/~widelands-dev/widelands/busy_roads_for_buildings/+merge/249013
Your team Widelands Developers is subscribed to branch
lp:~widelands
Forgotton: I want to store the related gimp-xcf files also somewhere in the
project. I think widelands-media is the correct place for that... but its quity
messy and i don't know where to put such files.
--
https://code.launchpad.net/~widelands-dev/widelands/busy_roads_for_buildings/+merge/24901
Thanks chuck,
the zip file contains xcf-files for each tribe plus one with testpatterns. If
you find the testpatterns are useless, throw it away.
Because it isn't possible to make an attachment here, i provide a link to it:
http://home.arcor.de/frank.ue/widelands_roads/roads_tribes.xcf.zip
--
@SirVer: Could you please explain, if the "is"-value has influence about
growing of trees?
As an example: "dry" means currently "no buildings possible, but roads". "dry"
associates also to something like desert, where trees do not grow good. But i
assume the new terrain affinitie-values have on
Review: Needs Fixing
> Yes, but that is no change in behavior to before though -
This wasn't my point. I don't want to get terrain developers on the wrong path
with ambiguous words. :-) "green" associates with growing plants, "dry" not.
But these values haven't anything to do with growing. Thi
Next week i provide more road patterns for each tribe. Each tribe will then
consist of minimum two patterns for normal roads.
--
https://code.launchpad.net/~widelands-dev/widelands/remove_concept_graphics/+merge/249770
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/wide
Because "dead" is only applied to greenland "snow" terrain, see
http://bazaar.launchpad.net/~widelands-dev/widelands/terrain_doc/view/head:/world/terrains/init.lua#L265
i suggest to remove "dead".
--
https://code.launchpad.net/~widelands-dev/widelands/terrain_doc/+merge/249747
Your team Widelan
I am further unhappy about the descriptions in terrains/init.lua, see below.
If someone want to make more terrains, he should be able to choose the right
"is" value. Call me scrupulously, sometimes i am ;)
Diff comments:
> === modified file 'src/ai/ai_help_structs.h'
> --- src/ai/ai_help_struct
Some wikipages do throw an internal server error if i try to edit them. Don't
know if it is related to this changes...
f.e.:
https://wl.widelands.org/wiki/Main%20Page/
https://wl.widelands.org/wiki/GameHelpbasicInterface/
https://wl.widelands.org/wiki/EditorHelpHelp/
https://wl.widelands.org/wiki
As SirVer said in
https://code.launchpad.net/~widelands-dev/widelands/terrain_doc/+merge/249747/comments/618862
"only terrain affinity decides how trees grow."
I think before the one-world merge, the is-values did have influence about the
growing of trees. With the merge into one-world, this te
> @kaputtnik: as you are the most active dev on the website right now, would
> you be okay with me adding you to the group that receives email notifications
> with backtraces about errors?
Yes.
> Should I use the email you signed up with on wl.widelands.org?
Yes.
Could i get an acc
Review: Approve
Thanks,
> Is this branch good to go now?
I think so. Thanks for your work :-)
--
https://code.launchpad.net/~widelands-dev/widelands/terrain_doc/+merge/249747
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/terrain_doc.
___
Thanks,
>> Could i get an account for the admin panel too?
> Done.
This doesn't work. I just see nothing, except that i do not have the rights to
change anything. Maybe i have to wait a while?
I coudn't trigger the error on my maschine. But i assume that this error occurs
with articles that co
> You can try to reproduce it on alpha.widelands.org
Over there the error occurs. I didn't found an article with newly uploaded
images, so i tried to create a new one and to upload an image. But then the
same failure happens as when i try to set up the website on my maschine:
No such file or di
After searching around i found this: http://stackoverflow.com/a/11346592
"As a note for other this issue can be caused when you have a double '//' in
the static file resource you are looking for."
The double slashes in our media path is something what i noticed erlier but i
was unsure about how
> It seems to be this change then. No more errors all day today.
Sorry, i do not understand. But maybe i also do not explain as much. Just to be
clear:
As you revert the changes, all is fine. No more errors, all articles could be
edited. But on the alpha sites, the error is allways there but i c
Thanks :-)
> also fixed the serving paths for static media, so that image uploading
> actually works. I tried one picture real quick, but could not trigger the
> error: http://alpha.widelands.org/wiki/edit/TestPageWithImage/
Ok, that article works. I uploaded an image there too and this aricle
kaputtnik has proposed merging lp:~franku/widelands-website/fix-double-slashes
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/fix-double-slashes/+merge/250231
This branch fixes the
Thanks for the dump.
> It is easily possible that it references broken data.
Ok...
> admin site: I thought I gave you access? I gave you full access now on alpha
> too.
For alpha it works, for widelands.org not. I just see nothing, except that i do
not have the rights to change anything.
> I t
> you have a lot of rights now - please use them wisely :)
I will change nothing, until i do really know what i am doing :-)
The exception raises in
/wlwebsite/lib/python2.7/site-packages/django/core/files/storage.py line 234.
There is a function "safe_join()" called which could be found in
/w
Sorry...
> First, how do I trigger the error on alpha - I have never seen it there.
Try http://alpha.widelands.org/wiki/edit/BarbariansPage/ or
http://alpha.widelands.org/wiki/edit/ImperialsPage/ These pages do trigger the
Suspicious-error. It should not matter, if the files not exists, because
Thanks for spending time to investigate this issue further :-)
A short conclusion:
> ('#sirver name: ',
> u'/var/www/django_projects/wlwebsite/code/widelands/media//wlimages/WL_barbarian_tribe_1024x768middle2')
Here is the file extension ".png" missing. This is maybe caused by he length of
the
Oh, i havn't seen this is deployd and sorry for testing :-(
The file "pip_requirements.txt" on
http://bazaar.launchpad.net/~widelands-dev/widelands-website/trunk/files is
still the same (no django 1.3.7) and its still on revision 383.
--
https://code.launchpad.net/~hjd/widelands-website/dj
The proposal to merge lp:~franku/widelands-website/handle_big_images into
lp:widelands-website has been updated.
Status: Merged => Work in progress
For more details, see:
https://code.launchpad.net/~franku/widelands-website/handle_big_images/+merge/247235
--
Your team Widelands Developers i
kaputtnik has proposed merging lp:~franku/widelands-website/forum_tables into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/forum_tables/+merge/250839
This branch make some corrections
kaputtnik has proposed merging
lp:~franku/widelands-website/fix-redirect-links-coloring into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/fix-redirect-links-coloring/+merge/251168
I am currently working on this and want to get your opinion on this behavior:
> I added code to change each external image link in a clickable image link, if
> it is not yet a clickable link. This has a disadvantage: Little images gets
> clickable too. But i think this is acceptable. Advantage:
Sorry for confusing you ... :-S
>> I added code to change each external image link in a clickable image link,
>> if it is not yet a clickable link.
> I think that is a good solution. If it is annoying in practice will be seen
> over time.
This was the answer i want to read ;) The Problem i saw
It's hard to explain what i meant what is happend to trigger the error. Fell
free to ask if something is unclear...
All following is related to widelands.org, not to the alpha-sites.
Fact is:
1. All images which was uploaded in year 2009 have the faulty path
(".../django_projects/widelands/medi
kaputtnik has proposed merging
lp:~franku/widelands-website/changes_to_admin_wlimages into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/changes_to_admin_wlimages/+merge/252996
This
kaputtnik has proposed merging
lp:~franku/widelands-website/forum_notifyings_contain_links_to_post into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website
Sorry, there went something wrong... i've updated this branch.
--
https://code.launchpad.net/~franku/widelands-website/forum_notifyings_contain_links_to_post/+merge/253003
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/forum_notifyings_con
Thanks for your review :-) . I just used the current text, reorder and put the
links inside.
I'll change the branch to your suggestions. While the changes affect only to
added posts, i use "was added to the topic" instead of "A new topic "{{ topic
}}" has been added to the forum."
Diff com
.body }}. The result will be, that the post is
displayed with full syntax, like as you write in the forum. Advantage: Quotes
are better displayed, f.e.:
"kaputtnik" wrote:
> *someone wrote:*
>
> some text
>
> Heres is a picture:  Thanks
--
https://code.launchpad.net/~franku/widelands-website/changes_to_admin_wlimages/+merge/252996
Your team Widelands Developers is subscribed to branch lp:widelands-website.
___
Mailing list: https://launchpad.net/~widelands-dev
P
I will split it into two branches. One containing the css-things and one with
the admin action.
--
https://code.launchpad.net/~franku/widelands-website/handle_big_images/+merge/247235
Your team Widelands Developers is subscribed to branch lp:widelands-website.
___
I splitted it into three branches:
The branch
https://code.launchpad.net/~franku/widelands-website/handle_great_images
contains the css related things and the stuff to wl_markdown.py for handling
big images which break the layout
The branch
https://code.launchpad.net/~franku/widelands-website
kaputtnik has proposed merging lp:~franku/widelands-website/handle_great_images
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/handle_great_images/+merge/254988
In continuation of
kaputtnik has proposed merging lp:~franku/widelands-website/correct_image_paths
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/correct_image_paths/+merge/254990
This branch
kaputtnik has proposed merging
lp:~franku/widelands-website/admin_action_correct_image_path into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/admin_action_correct_image_path/+merge
I looked into it and think its a good solution. Next week i will try to
implement it in the Website.
Thanks :-)
--
https://code.launchpad.net/~widelands-dev/widelands/authors/+merge/250513
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/au
kaputtnik has proposed merging lp:~franku/widelands-website/developers_list
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1424072 in Widelands Website: "Automatically add translator credits to
authors list"
https://bugs.lau
Oh sorry, this branch has to be merged only together with
https://code.launchpad.net/~widelands-dev/widelands/authors (if it is ready :-D
)
--
https://code.launchpad.net/~franku/widelands-website/developers_list/+merge/259041
Your team Widelands Developers is requested to review the proposed mer
kaputtnik has proposed merging lp:~franku/widelands-website/faulty_encyclopedia
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/faulty_encyclopedia/+merge/259967
Because from time
Thanks :-)
Text changed according to your suggestion.
--
https://code.launchpad.net/~franku/widelands-website/faulty_encyclopedia/+merge/259967
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/faulty_encyclopedia into lp:widelands-website.
Thanks :-)
I used this admin action to correct the paths of all images.
If you want to, you could remove this admin-action now... it isn't needed
anymore.
--
https://code.launchpad.net/~franku/widelands-website/admin_action_correct_image_path/+merge/254993
Your team Widelands Developers is requ
Yes :-)
--
https://code.launchpad.net/~franku/widelands-website/handle_great_images/+merge/254988
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/handle_great_images into lp:widelands-website.
I thank you for taking your time :-)
--
https://code.launchpad.net/~franku/widelands-website/handle_great_images/+merge/254988
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/handle_great_images into lp:widelands-website.
_
kaputtnik has proposed merging
lp:~franku/widelands-website/devs_and_locales_list into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1424072 in Widelands Website: "Automatically add translator credits to
authors list&quo
Thanks for approve :-)
Something i didn't found a rule for yet: When to use double quotes(") and when
to use single quotes (') ... I used double quotes anywhere. Maybe it would be
nicer to use single quotes on the keywords.
If we have some time, i will try to optimize the code... especially the
Related to ' vs. " its just programmers decision, but for one project it should
be consistent. Maybe SirVer could tell his preferred syntax. I changed it now
to something like 'vars/keywords that already exist' vs "new vars/real strings".
break statement is a good idea, i changed the code accord
After running pyformat over the file there some changes to old code also. I run
a last test of the file and all is good. So i think it's ready to merge.
Thanks for you hints :-)
--
https://code.launchpad.net/~franku/widelands-website/devs_and_locales_list/+merge/266025
Your team Widelands Develo
kaputtnik has proposed merging
lp:~franku/widelands/adjust_rolling_of_winterwaves into lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1408712 in widelands: "Waves are rolling in different speed /rolling
angle is different&quo
kaputtnik has proposed merging lp:~franku/widelands/better_dither_values into
lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands/better_dither_values/+merge/266673
This branch contains edited "dither
No, those values have only visual effects for the terrains which adjoin. See
https://wl.widelands.org/wiki/HelpTerrains/#dithering for consequences of this
value. I tried former to adjust them, but this attempt wasn't a success to
wasteland terrains. See https://bugs.launchpad.net/widelands/+bug
Could i delete this branch without breaking some stuff? This branch was merged
and reverted after some errors. (see:
https://code.launchpad.net/~franku/widelands-website/handle_big_images/+merge/247235/comments/619163)
The reason for deleting is just: I want to clean up :-)
--
https://code.laun
> If you set it's status to "merged" it will no longer be listed in most
> listings.
That's my problem: It is not merged. Merged are the new branches related to
this issue.
But i will leave it as is, because of the history.
Thanks for your answer :-)
--
https://code.launchpad.net/~franku/wide
Thanks, i changed it into "abandoned" in the branch details and the merge
status to "Rejected".
--
https://code.launchpad.net/~franku/widelands-website/handle_big_images/+merge/247235
Your team Widelands Developers is subscribed to branch lp:widelands-website.
___
The proposal to merge lp:~franku/widelands/new_tree_values into lp:widelands
has been updated.
Description changed to:
This branch contains the new values from king_of_nowhere for growing of trees.
Edit: See https://wl.widelands.org/forum/topic/1760/?page=9#post-14345 and the
whole thread.
Fo
kaputtnik has proposed merging lp:~franku/widelands/new_tree_values into
lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands/new_tree_values/+merge/267989
This branch contains the new values from
kaputtnik has proposed merging
lp:~franku/widelands-website/new_legal_notice_page into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1484845 in Widelands Website: "Add an Immpressum (legal) site"
https://bugs.launchpad.net
Do you mean to use a tuple like ('name', 'emailaddress')?
This should be doable.
--
https://code.launchpad.net/~franku/widelands-website/new_legal_notice_page/+merge/268973
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/new_legal_notice_p
Now INQUIRY_RECIPIENTS is used to have ('name','email address'). The email part
is not checked for validity.
I changed also the code to gather the "message" information.
--
https://code.launchpad.net/~franku/widelands-website/new_legal_notice_page/+merge/268973
Your team Widelands Developers is
kaputtnik has proposed merging lp:~franku/widelands-website/css_changes into
lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/css_changes/+merge/271388
This branch contains changes to
Please wait with merging, there are some changes and i will update the branch
soon.
--
https://code.launchpad.net/~franku/widelands-website/css_changes/+merge/271388
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/css_changes into lp:widel
Uodated... damn, there is unused code. I have to update again :-S
--
https://code.launchpad.net/~franku/widelands-website/css_changes/+merge/271388
Your team Widelands Developers is requested to review the proposed merge of
lp:~franku/widelands-website/css_changes into lp:widelands-website.
Updated.
Changes to text-shadow are reverted, so text-shadow is like the current state.
Additional changes: The menu entries are font-size dependent in width. So if
someone changes the font size in his browser, the width of the entries gets
adjusted too.
I provide screenshots in the forum topi
Thanks, all works as suspected :-)
I think no one would notice the changes... but if there are any unwanted
changes, please report in the forum topic:
https://wl.widelands.org/forum/topic/1797/
--
https://code.launchpad.net/~franku/widelands-website/css_changes/+merge/271388
Your team Widelan
kaputtnik has proposed merging lp:~franku/widelands-website/little_navi_changes
into lp:widelands-website.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~franku/widelands-website/little_navi_changes/+merge/276625
Just a few changes
Review: Needs Fixing
Because luatables have changed this needs fixing.
There is also a conflict when merging with trunk.
I'll take a look at this.
--
https://code.launchpad.net/~widelands-dev/widelands/artifacts/+merge/276527
Your team Widelands Developers is subscribed to branch
lp:~widelands
Review: Resubmit
Should work now again :-)
--
https://code.launchpad.net/~widelands-dev/widelands/artifacts/+merge/276527
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/artifacts.
___
Mailing list: https://launchpad
Review: Needs Fixing
If there is nothing against it, i would change the files to have the
translations back again (place an underscore in the files by hand). So we could
merge this branch.
Nevertheless the bug 1514342 still exists, also in current trunk.
--
https://code.launchpad.net/~wideland
Review: Resubmit
The files are changed.
--
https://code.launchpad.net/~widelands-dev/widelands/artifacts/+merge/276527
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/artifacts.
___
Mailing list: https://launchpad.ne
Review: Needs Fixing
Sorry, there is more lost:
When loading/saving a map the suggested teams gets lost also. So this branch do
not suggest teams if you want to play a map. F.e. map Elven Forest
--
https://code.launchpad.net/~widelands-dev/widelands/artifacts/+merge/276527
Your team Widelands D
Loosing the suggested teams happens also in current trunk. So your changes to
the artifact branch do not cause the problem. I guess that making suggested
teams was formerly done by editing the file "elemental" manually because we
haven't had such an option in the map options menu. May this issue
There where only 7 maps which have suggested teams. Done fixing suggested teams
:-)
--
https://code.launchpad.net/~widelands-dev/widelands/artifacts/+merge/276527
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/artifacts.
__
Review: Resubmit
--
https://code.launchpad.net/~widelands-dev/widelands/artifacts/+merge/276527
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/artifacts.
___
Mailing list: https://launchpad.net/~widelands-dev
Post
kaputtnik has proposed merging lp:~widelands-dev/widelands/worker_icons into
lp:widelands.
Requested reviews:
Widelands Developers (widelands-dev)
Related bugs:
Bug #1512077 in widelands: "change icons for workers in the menus"
https://bugs.launchpad.net/widelands/+bug/1512077
kaputtnik has proposed merging lp:~widelands-dev/widelands/is_value_terms into
lp:widelands.
Commit message:
Changing the is values in world/terrains/init.lua and the terms for is-value
related things in C++ code.
dead -> unreachable
mountain -> minebale
green -> arable
dry -&
The proposal to merge lp:~widelands-dev/widelands/is_value_terms into
lp:widelands has been updated.
Commit Message changed to:
Changing the is values in world/terrains/init.lua and the terms for is-value
related things in C++ code.
dead -> unreachable
mountain -> minebale
green -> arable
dry
Sorry that i couldn't approve your code... i could only say it works :-D
--
https://code.launchpad.net/~widelands-dev/widelands/is_value_terms/+merge/279205
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/is_value_terms.
> pics/ai_strong.png is now unused
I could put it into the media branch.
Currently i compile this branch and try to test the changes. After i am ready i
will update this branch (removing the png file) and update the media branch
also.
--
https://code.launchpad.net/~widelands-dev/widelands/ai_l
I could confirm the fuzzy roads. Additionally the buildings menu is messed up.
I put a screenshot into the bug 1425870.
--
https://code.launchpad.net/~widelands-dev/widelands/render_queue/+merge/250524
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/render_queu
mode:
terminate called after throwing an instance of 'ImageNotFound'
what():
[/home/kaputtnik/Quellcode/widelands-repo/ai_level_names/src/graphic/image_io.h:35]
Image not found: pics/ai_Normal.png
I fixed this by editing scripting/widelands/multiplayer_help.lua and changed
string &qu
The proposal to merge lp:~widelands-dev/widelands/spinbox into lp:widelands has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/spinbox/+merge/279153
--
Your team Widelands Developers is requested to review the propos
I've tested the option window and the spin boxes with resolution 800x600 and
1024x768.
Looks good :-)
--
https://code.launchpad.net/~widelands-dev/widelands/spinbox/+merge/279153
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/spinbox into
I am not sure where the Ware help could be found :-S *lol*
Open the tribal help by hitting the questionmark on the bottom i found two
things:
1. The worker icon in the tab is the old icon. I think we should use the same
icons for workers in all menus. Somewhere i found also the old one, but i d
I will test your changes tomorrow.
We should have an overlook on the whole icons i believe. I feel that there are
some icons which are either not used or could be replaced with other existing
icons... hm... time... give me time...
--
https://code.launchpad.net/~widelands-dev/widelands/ware_help
The proposal to merge lp:~widelands-dev/widelands/ware_help into lp:widelands
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/ware_help/+merge/276516
--
Your team Widelands Developers is subscribed to branch
lp:~
Review: Approve
Looks good :-)
One thing: In the wares help and in workers help the deep mines comes before
mines. F.e. for ware "coal"
I will approve nevertheless. You could self decide if you want to fix this in
here.
--
https://code.launchpad.net/~widelands-dev/widelands/ware_help/+merge/2
Sorry.
--
https://code.launchpad.net/~widelands-dev/widelands/ware_help/+merge/276516
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/one_tribe.
___
Mailing list: https://launchpad.net/~widelands-dev
Post to : w
1 - 100 of 1276 matches
Mail list logo
