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

Fixes misses closing '/' in component declaration #278

Merged
merged 1 commit into from
Mar 2, 2017

Conversation

Worthaboutapig
Copy link
Contributor

The component declaration 'Component' is missing a now-required '/' on the tag. As is stands the example fails.

Is there not a better way of ensuring that documentation samples stay up-to-date and correct? As it's the first port-of-call for people learning a feature, if the examples are wrong, it doesn't create a good experience, especially as one assumes that one's own code is wrong, rather than the example.

Maybe embed a fiddle instead of the raw code? Then this can be easily tested on each new release.

The component declaration 'Component' is missing a now-required '/' on the tag.  As is stands the example fails.

Is there not a better way of ensuring that documentation samples stay up-to-date and correct?  As it's the first port-of-call for people learning a feature, if the examples are wrong, it doesn't create a good experience, especially as one assumes that one's own code is wrong, rather than the example.

Maybe embed a fiddle instead of the raw code?  Then this can be easily tested on each new release.
@Worthaboutapig
Copy link
Contributor Author

Something like: https://jsfiddle.net/worthaboutapig/jhrgcmLk/ in which I've tried to make the code better explain the setting... at least I think so ;)

@Worthaboutapig
Copy link
Contributor Author

I see the fiddle thing has been mentioned in #261. #262, however can we NOT add stuff that isn't tested? As having the docs wrong is the worst thing IMHO.

@fskreuz
Copy link
Contributor

fskreuz commented Mar 2, 2017

Documentation and other sections of the ractivejs.org domain is currently being migrated and merged over to https://ractive.js.org/. The new repo is now at https://github.com/ractivejs/ractivejs.github.io. We will sunset this repo and site along with the others when we get to it.

As having the docs wrong is the worst thing IMHO.

Yep, I agreed 102%. The typo might still exist in the new repo though. I still have to review all the verbiage and links when I get the time, even for the legacy docs. 😄

@fskreuz fskreuz merged commit a96d8d4 into ractivejs:master Mar 2, 2017
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.

2 participants