search

herosheets / woin-starships

Starship Creator for WOIN
http://herosheets.github.io/woin-starships/
1 stars 2 forks source link

Separate power plant block from computer display #58

Closed herosheets-codeship closed 8 years ago

herosheets-codeship commented 8 years ago

In Computers there's a section of Power Plants. That should really be its own tab and line in the stat block.

Morrus commented 8 years ago

I think if we could quickly slip this one in, we can open it up to beta testing.

jsomara commented 8 years ago

I see the new sections in the CSVs. You say this should be its own tab, meaning you'd like a separate power plant section beneath Command & Control in the left navigation?

Can a user select both a computer and a power plant?

What about the Navigation Computers, AI and Fasa Trek - which tab will these end up on? The first two sound like sub-categories of "Computers"

Assuming you want these as a separate section, go ahead and make a powerplants.csv file and move the appropriate rows over there. Then I would add a "category" column to each, and put the appropriate category in that column (AI, Fasa Trek, etc)

Morrus commented 8 years ago

Yeah, just the power plants bit - the others are just types of control computer split up to ease navigation. No need to worry about doing anything to navigation computers, AI, and FASA Trek.

You can select both computer and power plant. Originally they were supposed to give different resources (CPU and Power) but they got smooshed into just CPU.

Sent from my iPhone

On 2 Nov 2015, at 01:45, Jordan OMara notifications@github.com wrote:

I see the new sections in the CSVs. You say this should be its own tab, meaning you'd like a separate power plant section beneath Command & Control in the left navigation?

Can a user select both a computer and a power plant?

What about the Navigation Computers, AI and Fasa Trek - which tab will these end up on? The first two sound like sub-categories of "Computers"

Assuming you want these as a separate section, go ahead and make a powerplants.csv file and move the appropriate rows over there. Then I would add a "category" column to each, and put the appropriate category in that column (AI, Fasa Trek, etc)

— Reply to this email directly or view it on GitHub.

jsomara commented 8 years ago

Got it - I just split these out and pushed it, so just create powerplants.csv with the info you'd like and it should work. You'll need to edit the helper text however you see fit as well, it's copied from Computers.

Morrus commented 8 years ago

Cool. Done! I renamed the table headers on that new HTML file, too, to say Power Plants -- I don't know if that will affect anything, but if it breaks, that will probably be why!

Sent from my iPad

On 2 Nov 2015, at 02:05, Jordan OMara notifications@github.com wrote:

Got it - I just split these out and pushed it, so just create powerplants.csv with the info you'd like and it should work. You'll need to edit the helper text however you see fit as well, it's copied from Computers.

— Reply to this email directly or view it on GitHub.

Morrus commented 8 years ago

Yeah, it didn’t like me changing the headers on the table to Power Plant! At least, I think that’s what happened – the power plants file isn’t working correctly, but I’m pretty sure that’s because of me. Hopefully something you can fix easily!

From: Jordan OMara Sent: Monday, November 2, 2015 2:04 AM To: herosheets/woin-starships Cc: Morrus Subject: Re: [woin-starships] Separate power plant block from computer display (#58)

Got it - I just split these out and pushed it, so just create powerplants.csv with the info you'd like and it should work. You'll need to edit the helper text however you see fit as well, it's copied from Computers.

— Reply to this email directly or view it on GitHub.