Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Table Documentation #168

Open
wants to merge 11 commits into
base: gh-pages
Choose a base branch
from
Open

Conversation

dragonexpert
Copy link
Contributor

Broke down all the tables to show the columns and what they are used for.

@euantorano
Copy link
Member

This is a big one! I wonder if it might make sense to make the table names bold and differentiate the column names from the descriptions somehow - perhaps by making the column name italic, or maybe by using a HTML table to show each table?

@dragonexpert
Copy link
Contributor Author

So maybe The table name on top. Then th for column name, column type, description? If so, it would be best to build a script to do that as it would take a lot longer to do manually.

@euantorano
Copy link
Member

euantorano commented Feb 12, 2018 via email

@dragonexpert
Copy link
Contributor Author

Here is a preview of what I have: http://superlatios.com/forums/totable.html I think we should assign a fixed width for each table though. Just let me know what to put so I can generate it.

@andrewjs18
Copy link
Contributor

looks pretty good, @dragonexpert. I agree with you on a fixed width.

@dragonexpert
Copy link
Contributor Author

Do you think 1000px would be sufficient? @euantorano @andrewjs18

@andrewjs18
Copy link
Contributor

andrewjs18 commented Feb 18, 2018

probably would be fine...most people use a resolution above 1024 these days anyways.

@dragonexpert
Copy link
Contributor Author

I'll generate the tables with a fixed width tomorrow then and update the PR.

@Ben-MyBB
Copy link
Member

We should probably try to get this merged. @andrewjs18 @euantorano @justinsoltesz ?

@euantorano
Copy link
Member

Yep, we should. I'm not sure how up to date this PR is, but we could merge it and update it I guess.

@andrewjs18
Copy link
Contributor

bump, @dragonexpert

@Ben-MyBB
Copy link
Member

Ben-MyBB commented May 3, 2020

@dragonexpert any chance you could update this one?

@Ben-MyBB
Copy link
Member

@yuliu is there any chance you can check the content here?

@yuliu
Copy link
Member

yuliu commented May 19, 2020

@yuliu is there any chance you can check the content here?

Sure I'll take a look in this weekend. But what if the content is out of date? Database changes exist in recent version updates.

@euantorano
Copy link
Member

It probably is out of date, but we can always merge then update or update the PR directly (it appears to be open to maintainers supplying changes directly).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants