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

move website into main repo #931

Closed
wants to merge 29 commits into from
Closed

move website into main repo #931

wants to merge 29 commits into from

Conversation

aciccarello
Copy link
Collaborator

This moves the TypeStrong/typedoc-site repo into this project to make it easier to update the API docs and for contributors to find the website source.

See the changes since 90bc192 for the changes to the website.

Closes #604
Closes #658
Closes TypeStrong/typedoc-site#6
Closes TypeStrong/typedoc-site#8

d-asensio and others added 29 commits August 26, 2016 01:01
The .nojekyll file didn't work b/c it needs to be in the root. These changes are untested because Windows...
Updating doccomments to include a few other javadoc tags
git-subtree-dir: website
git-subtree-mainline: 6d5b21b
git-subtree-split: d1e3895
used on guide layout

closes typedoc/typedoc-site#8
reverted rename of Installation page to Usage
Copy link
Contributor

@9oelM 9oelM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked several parts of the website. Focused on fixing things that are supposed to work and adding possibly useful info.

height: 100%
background-image: url(../images/jumbotron.png)
background-position: center center

Copy link
Contributor

@9oelM 9oelM Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @aciccarello, I have made a build with jekyll and am looking into the website.

It was always a pain to see typedoc "visual" cut on the edge on my phone, like:
cut

If you add

Suggested change
background-size: cover

It's going to be 200% prettier!
notcut

@@ -0,0 +1,12 @@
<header>
<div class="container">
<a href="{{ site.baseurl }}" class="logo">TypeDoc</a>
Copy link
Contributor

@9oelM 9oelM Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this line produces

 <a href="" class="logo">TypeDoc</a>

and does nothing if you click on it. And this is what's actually happening in the current typedoc website. I know baseurl is defined in _config.yml but it seems you cannot access it by writing {{ site.baseurl }} (official doc appears to have no site.baseurl).

If you have made it an <a> tag, I suspect it's intended to redirect to the index page.
If you meant so, we could just

Suggested change
<a href="{{ site.baseurl }}" class="logo">TypeDoc</a>
<a href="/" class="logo">TypeDoc</a>

<title>{{ page.title }} | TypeDoc</title>
<meta name="description" content="A documentation generator for TypeScript projects." />
<meta name="keywords" content="TypeScript documentation generator javascript open source" />

Copy link
Contributor

@9oelM 9oelM Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps add some og tags? It will help typedoc gain more exposure and be shared more easily.

Suggested change
<!--facebook-->
<meta property="og:url" content="{{ page.url | absolute_url }}">
<meta property="og:description" content="A documentation generator for TypeScript projects.">
<meta property="og:title" content="{{ page.title }}">
<meta property="og:type" content="website" />
<meta property="og:site_name" content="{{ site.title }}">
<meta property="og:image" content="{{ "/images/logo-256.png" | absolute_url }}" />
<!--google+-->
<meta itemprop="name" content="{{ page.title }}">
<meta itemprop="description" content="A documentation generator for TypeScript projects.">
<!--Twitter-->
<meta name="twitter:card" content="{{ site.title }}">
<meta name="twitter:url" content="{{ page.url | absolute_url }} ">
<meta name="twitter:title" content="{{ page.title }}">
<meta name="twitter:description" content="A documentation generator for TypeScript projects.">

All comments are parsed as markdown. TypeDoc uses the [Marked](<https://github.com/chjj/marked>) markdown parser
and [HighlightJS](<https://github.com/isagalaev/highlight.js>) to highlight code blocks within markdown sections.
Additionally you can link to other classes, members or functions using double square brackets.

Copy link
Contributor

@9oelM 9oelM Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aciccarello are you going to add a note about #917 as well here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to. I wasn't sure if I wanted to do that in this PR or a follow up one. If I have time I'll add it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool!

@9oelM
Copy link
Contributor

9oelM commented Dec 29, 2018

@aciccarello
@cancerberoSgx

Is there any chance #11 in typedoc-site repo will come alive together? It seems to be a really good resource.

```

Exclude files by the given pattern when a path is provided as source

Copy link
Contributor

@9oelM 9oelM Dec 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a note on excluding multiple patterns, as many people wonder how. As of now, the best practice appears to be put an array of strings of patterns to exclude inside typedoc.js as specified by @dupski in #170.

Suggested change
Multiple files could also be excluded by adding an array of strings in the `exclude` key of `typedoc.js` like this:
```js
module.exports = {
out: './docs/dist/api/',
readme: 'none',
includes: './',
exclude: [
'**/__tests__/**/*',
'**/__test_utils__/**/*',
'**/__fixtures__/**/*',
'**/testsuite/**/*'
],
mode: 'file',
excludeExternals: true,
excludeNotExported: true,
excludePrivate: true
};
```

Copy link
Collaborator

@Gerrit0 Gerrit0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really need to include the generated files in both this repo and the typedoc-site repo? I'd rather not require checkouts to pull in so many generated html files.

Plugins are available for Grunt, Gulp, and Webpack. All of them can be installed using ``npm``.


### Grunt
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was under the impression that no grunt plugin was required for typedoc, why else do we have tasks/typedoc.js?

I don't use grunt anywhere else, but the linked plugin hasn't been updated in years.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's been in the docs for years. I assume the grunt plugin still works and I think it'd be worth leaving it in there if it does.

@9oelM
Copy link
Contributor

9oelM commented Dec 31, 2018

@Gerrit0 What do you exactly mean by the generated files? Do you mean the ones for typedoc api generated by typedoc under website/api/?

@Gerrit0
Copy link
Collaborator

Gerrit0 commented Dec 31, 2018

Yes, if all the html pages are being pushed to another repo, I don't see the need to track them here as well.

@9oelM
Copy link
Contributor

9oelM commented Dec 31, 2018

You are right. we do not really need to include those htmls in website/api. Actually we could write some short travis scripts to automatically typedoc --out website/api/ src/index.ts on cloud and deploy the page to github pages, without needing to track them anywhere. This is going to keep the documentation up to date every time the website is deployed, which is going to automate the process as well.

And doesn't @aciccarello have a plan to archive/delete typedoc-site?

@aciccarello
Copy link
Collaborator Author

👍 Yeah, we shouldn't have the generated api files in here.

ATM, I'm planning on using gh-pages to push to the typedoc-site repo. If we want to move things to a gh-pages branch (which might make sense) we can do that later. But I don't have access to the website hosting so I didn't want to change any of that.

@Gerrit0
Copy link
Collaborator

Gerrit0 commented Dec 31, 2018

You should be able to change the website settings - I can see, and presumably change gh-pages settings here: https://github.com/TypeStrong/typedoc-site/settings

@aciccarello
Copy link
Collaborator Author

I knew about some of that config but assumed there would be some DNS config I couldn't access. We can look into it following this PR.

@Gerrit0
Copy link
Collaborator

Gerrit0 commented Dec 31, 2018

All GitHub pages sites use the same DNS configuration, so no changes should be required there. I agree - that change can wait until after the site is complete.

@9oelM
Copy link
Contributor

9oelM commented Jan 8, 2019

Any updates?

@cancerberoSgx
Copy link

@9oelM @aciccarello hello, I can review the tutorial and perhaps remove a couple of sections that were kind of a research, like the event emitter one or make it shorter, I really don't have much time to invest on this, but if you think it will be merged so the doc is public I can make some time for this :P Somebody told me that this project is kind of obsolete in favor of other (i think it was ts-doc from microsoft) but I tried it some month ago and is not as mature as typedoc (features and ecosystem) . Is this right or I misunderstood, what's the situation here so is worthwhile to invest on this ? Thanks and keep it up, I think with a little more documentation this tool is great, even compared to jsdoc (JavaScript's) ones.

@aciccarello
Copy link
Collaborator Author

@9oelM @cancerberoSgx I still plan on updating this based on the feedback. From there we can merge this update and move towards improving the docs. Eventually I would like to add the tutorial @cancerberoSgx worked on but this is a first step to making it easier to work on the website.

@cancerberoSgx
Copy link

Nice! will try to review the tutorial and remove research-sections and references to some plugins that I think the tutorial adds, so is more basic, thanks

@cancerberoSgx
Copy link

@aciccarello should I keep working on typedoc--site PR TypeStrong/typedoc-site#11 or do you think is better to start using this PR and integrate the tutorial here ? In the first case I wonder how difficult would be to re-integrate the changes in obsolete repo typedoc-site, thanks

@aciccarello
Copy link
Collaborator Author

@cancerberoSgx This PR is mostly a copy from typedoc-site but it is is probably easiest to work off of this PR

@Gerrit0 Gerrit0 mentioned this pull request Jan 21, 2019
9 tasks
@aciccarello
Copy link
Collaborator Author

Thanks for all the feedback. I've decided to not move the content into this repo to keep things simpler. This work has been continued in TypeStrong/typedoc-site#12.

We should update the contributing guide to link to the content.

@aciccarello aciccarello closed this Feb 8, 2019
@Gerrit0 Gerrit0 deleted the add-docs branch January 14, 2020 20:05
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.

8 participants