kind of new here

Here, various developers can tell you all about what they're up to, so you can yell at them for being idiots. "... and there was a great rejoicing."
Post Reply
Message
Author
stpa
Space Krill
Posts: 4
Joined: Mon Mar 12, 2018 1:08 pm

kind of new here

#1 Post by stpa » Mon Mar 12, 2018 1:30 pm

Hi. A long time ago (I think at least 3 years) I added that right click context menu for galactic map and nothing else and then disappeared. Now I'm back. First thing I did now was to add a column to the census report with the number of worlds, see attached screenshots. Looking foreward to adding some more little things before eventually getting bigger things done … if I do not disappear again, that is.
Attachments
with-better-number-formatting.png
with corrected number formatting, so that the numbers better align
with-better-number-formatting.png (162.51 KiB) Viewed 303 times
with-bad-number-formatting.png
screenshot with new census column for populated worlds and two new labels at the bottom (and with bad number formatting for big numbers as it was before)
with-bad-number-formatting.png (133.63 KiB) Viewed 303 times
Last edited by stpa on Fri Mar 16, 2018 7:41 pm, edited 1 time in total.

User avatar
Oberlus
Small Juggernaut
Posts: 738
Joined: Mon Apr 10, 2017 4:25 pm

Re: kind of new here

#2 Post by Oberlus » Mon Mar 12, 2018 4:44 pm

Great, I like it.
Time to get in touch with the @devs, submit PRs and all that stuff.

stpa
Space Krill
Posts: 4
Joined: Mon Mar 12, 2018 1:08 pm

Re: kind of new here

#3 Post by stpa » Mon Mar 12, 2018 9:10 pm

Yes, they did with me ^^ and of course I release the screenshots in this thread and all code & other contributions under GPL v2 (or compatible licence, at the discretion of the FreeOrion team), and I promise to use better indentation in my coding and I will comply with branch naming conventions et cetera, and thank you for liking it

stpa
Space Krill
Posts: 4
Joined: Mon Mar 12, 2018 1:08 pm

Re: kind of new here

#4 Post by stpa » Wed Mar 14, 2018 6:51 pm

Next thing, I did a little in-game indentation fixing in addition to indentation in the codebase. The Planet suitability report looked broken, when resizing the Pedia-window it seemed like too many tab signs \t, like in the attached screenshot. Which annoyed me, because in the official download 0.4.7.1 (build 2017-09-03.139ffd9) which I was play(test)ing before it was not broken, so I fixed it. At least for me. No idea how that gets rendered on other platforms since I've got only a mac. But I guess the devs will know how to test on multiple platforms …
Attachments
Bildschirmfoto 2018-03-14 um 19.18.01.png
broken tabs / columns in planet suitability report
Bildschirmfoto 2018-03-14 um 19.18.01.png (178.62 KiB) Viewed 339 times

User avatar
Geoff the Medio
Programming, Design, Admin
Posts: 12290
Joined: Wed Oct 08, 2003 1:33 am
Location: Munich

Re: kind of new here

#5 Post by Geoff the Medio » Wed Mar 14, 2018 7:57 pm

stpa wrote:First thing I did now was to add a column to the census report with the number of worlds, see attached screenshots.
Headings indicating what the columns display would be helpful...

stpa
Space Krill
Posts: 4
Joined: Mon Mar 12, 2018 1:08 pm

Re: kind of new here

#6 Post by stpa » Sun Mar 18, 2018 10:11 am

I have added the name of the planet of origin to the worldtree special's accounting label for the happiness effect on other planets, which was missing before, and I just wanted to know where the effect came from. This also has helped me to get to know how speciels and effects are handled in general.
Attachments
worldtree_accounting_label_before.png
before the change
worldtree_accounting_label_before.png (106.24 KiB) Viewed 296 times
worldtree_accounting_label_after.png
with the change
worldtree_accounting_label_after.png (148.26 KiB) Viewed 297 times

User avatar
Dilvish
AI Lead, Programmer
Posts: 4736
Joined: Sat Sep 22, 2012 6:25 pm

Re: kind of new here

#7 Post by Dilvish » Sun Mar 18, 2018 3:08 pm

stpa wrote:I have added the name of the planet of origin to the worldtree special's accounting label for the happiness effect on other planets, which was missing before...
Yes, that's a good catch, please put up a PR for it.
If I provided any code, scripts or other content here, it's released under GPL 2.0 and CC-BY-SA 3.0

Post Reply