Skip to content

Conversation

cmontella
Copy link
Owner

No description provided.

@cmontella cmontella changed the title Add console and file Document console and file watchers Jun 28, 2017
## File
Watchers that access the filesystem. These watchers only work in headless mode.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"Watchers" -> "Libraries"

<li><strong>element</strong> - (optional) - the element which has focus during key-up or key-down</li>
<li><strong>key-code</strong> - the key code of the key to be monitored</li>
<li><strong>key</strong> - (optional) - the key to be monitored; an alternative to key-code for a premade list of keys:
<li><strong>key</strong> - (optional) - the key to be monitored; an alternative to key-code for a pre-made list of keys:
Copy link
Collaborator

Choose a reason for hiding this comment

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

"premade" is already a word, doesn't need a hyphen

</td>
<td>
<code>
Write a message to the console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"// ..."

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Write a warning to the console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"// ..."

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Write an error in the console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

"// ..."

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Read a file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Most comments in the code are preceded by "// ...", let's just make sure everything looks the same

Copy link
Owner Author

@cmontella cmontella Jun 28, 2017

Choose a reason for hiding this comment

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

I've got a branch that takes these out, and formats the examples as standard Eve. Going to make a PR for that soon. Putting the // isn't necessary outside a block

</td>
<td>
<code>
Write a string to a file
Copy link
Collaborator

Choose a reason for hiding this comment

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

"// ..."

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Display a message when a file does not exist
Copy link
Collaborator

Choose a reason for hiding this comment

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

"// ..."

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

Copy link
Collaborator

@joshuafcole joshuafcole left a comment

Choose a reason for hiding this comment

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

👍

</td>
<td>
<code>
Display a message when a file does not exist
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Write a message to the console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Write a warning to the console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Write an error in the console.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

</td>
<td>
<code>
Write a string to a file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not needed outside of the code ticks.

@cmontella cmontella merged commit 57912e6 into gh-pages Jun 28, 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.

3 participants