Author Topic: [WIP] [Suggestion] Show amount owned on manufacture screen  (Read 812 times)

Offline Hitnrun

  • Squaddie
  • *
  • Posts: 5
    • View Profile
[WIP] [Suggestion] Show amount owned on manufacture screen
« on: February 19, 2018, 08:57:29 pm »
Hello,

I don't know if this is the section to post source code changes, if not please move it to the correct place.

I implemented in the "new manufacture" OXCE+ screen a column showing the amount owned of each item, so you don't have to remember what item you need to build, specially ammo.

It is a sum of all bases, including in-craft.

Attached is a screenshot of it.

Here is the source code:
https://github.com/RangelReale/OpenXcom/commits/oxce3.5-plus-proto-manufcount

See if you like it, I can make a PR if so.

« Last Edit: July 17, 2018, 02:32:15 pm by Meridian »

Offline FeruEnzeru[RETIRED/I QUIT]

  • Colonel
  • ****
  • Posts: 430
  • retired, because of Meridian and his followers. 👎
    • View Profile
Re: [EXE] Show amount owned on manufacture screen
« Reply #1 on: February 19, 2018, 09:36:17 pm »
Hello, Hitnrun. Welcome to the OXC forums.

Nice to meet you. :)


Feel free to ask some people about your idea of showing amout owned on the "Manufacture" screen, maybe they all like it here. :)

Offline Meridian

  • Global Moderator
  • Commander
  • ***
  • Posts: 5186
  • Aaand we're back!
    • View Profile
    • My Wiki
Re: [EXE] Show amount owned on manufacture screen
« Reply #2 on: February 19, 2018, 10:56:40 pm »
Thanks.
I'll put it in the queue and have a look later.

PR is not necessary, I hate merging from all my heart, merge history is a plague.
Cherry-picking is infinitely better.

@all: feel free to comment if you would like to see this feature in OXCE+

Offline FeruEnzeru[RETIRED/I QUIT]

  • Colonel
  • ****
  • Posts: 430
  • retired, because of Meridian and his followers. 👎
    • View Profile
Re: [EXE] Show amount owned on manufacture screen
« Reply #3 on: February 19, 2018, 10:57:11 pm »
Thanks.
I'll put it in the queue and have a look later.

PR is not necessary, I hate merging from all my heart, merge history is a plague.
Cherry-picking is infinitely better.

@all: feel free to comment if you would like to see this feature in OXCE+
Agreed! ^^

Go ahead and add it. :)

Offline Solarius Scorch

  • Global Moderator
  • Commander
  • *****
  • Posts: 8648
  • WE MUST DISSENT
    • View Profile
    • Nocturmal Productions modding studio website
Re: [EXE] Show amount owned on manufacture screen
« Reply #4 on: February 19, 2018, 11:13:33 pm »
I definitely vote yes.

Offline Eddie

  • Colonel
  • ****
  • Posts: 458
    • View Profile
Re: [EXE] Show amount owned on manufacture screen
« Reply #5 on: February 19, 2018, 11:24:40 pm »
Yes, please add.