Opened 8 years ago

Closed 8 years ago

Last modified 5 years ago

#3744 closed enhancement (fixed)

[PATCH] Credits page

Reported by: Itms Owned by: Itms
Priority: Should Have Milestone: Alpha 20
Component: Non-game systems Keywords: patch
Cc: Erik Johansson Patch:

Description

Hi, my credits page patch should be ready now.

I'm still missing some names, I'm currently contacting people, but in the meantime this is ready for review.

The translators are a specific case: there is a Python script that extracts their names from the .po files. Please comment on this matter. The output formatting is hideous, which is a shame because my script allows us to manually add people to the file. I didn't add the output in the patch (because of the poor formatting it makes a huge file), but you can test the script and see how it works.

This script might have to be added to the maintenance tasks script, but I can't really test .sh scripts here.

Finally, when I commit this I propose to make a change to SubmittingPatches so that new contributors add a change to the programming.json file. That way they can choose themselves how to be credited and this makes things easier. Do you agree with the proposal?

Attachments (1)

credits.patch (50.0 KB ) - added by Itms 8 years ago.

Download all attachments as: .zip

Change History (12)

by Itms, 8 years ago

Attachment: credits.patch added

comment:1 by Lionkanzen, 8 years ago

No requieres autobuild?

comment:2 by Itms, 8 years ago

No, you don't need to build (but unless you run the python scripts there won't be anyone in the Translators list)

comment:3 by fabio, 8 years ago

Would be possible to avoid the use of the phyton script reading the translator names directly from the .po files through some js code?

comment:4 by Lionkanzen, 8 years ago

Ok , works fine, and is beatuful page.

comment:5 by leper, 8 years ago

At least the programmers list has a few "nick": "" or "name": "" entries which should probably be removed.

For the python script I guess it would be better to make that part of the release process instead. (So there is no need to duplicate the language list)

Not sure if it would make sense to read the po files from js.

JS code looks good, but note a few of my comments regarding musicians, used libraries, and used source files (eg CC-BY, CC-BY-SA which would fit in here nicely).

comment:6 by Itms, 8 years ago

I just don't know about not using a python script: that sounds like a good idea to me mods-wise but we won't be able to manually add elements, unless we hardcode them in the js file. And I don't know how to regex in js, so there's that :)

The empty nicks/names are just a personal remainder, I'll remove them when I finish going through sending emails to people. (see also the "Foo - " in the forum topic, they should be in sync)

Could you list (by PM if you want) the missing things so I add them? I don't really know where to find those pieces of information and I'm likely to forget things.

comment:7 by Itms, 8 years ago

I managed to get the email addresses of everyone missing. I will be away the upcoming week, when I come back I'll take a look at the answers I got and will look into committing.

comment:8 by Itms, 8 years ago

Resolution: fixed
Status: newclosed

In 17830:

New in-game Credits page, fixes #3744.

comment:9 by Itms, 8 years ago

Keywords: review removed

comment:10 by elexis, 8 years ago

In 18745:

Add click button sound to the gamesetup icon buttons and credits page. Patch by Vladislav, fixes #4222, refs #3805, #3744.

comment:11 by elexis, 5 years ago

In 22966:

Fix top panel being hidden during pause mode following rP22957 / D2292 reported by Freagarach, ValihrAnt, Krinkle et al.

Remove the false misleading default.cfg z value that is always overwritten by CGUI::Xeromyces_ReadObject, both introduced in rP141.
Remove multiline property since its redundant with the default C++ boolean, making the default style more agnostic of final GUI Object classes.

Close advanced locale dialog correctly upon cancel following rP22676 / D1684.
Don't switch the background image when closing the credits dialog following rP17830, refs #3744.

Note: See TracTickets for help on using tickets.