Skip to content
This repository has been archived by the owner on Nov 17, 2017. It is now read-only.

chart component docs #5

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

chart component docs #5

wants to merge 6 commits into from

Conversation

lmacko
Copy link
Contributor

@lmacko lmacko commented Sep 23, 2013

No description provided.

@bleathem
Copy link
Member

I made some grammatical corrections and merged into the branch "chart-docs":
https://github.com/richfaces/richfaces-docs/tree/chart-docs

@@ -6537,6 +6537,154 @@ Read this chapter for details on components that display messages and other feed

[[sect-Component_Reference-Output_and_messages-message]]

=== [sgmltag]+<r:chart>+

The [sgmltag]+<r:chart>+ component allows the user to plot data and to create line, bar or pie charts. It might uses five children tags [sgmltag]+<r:series>+, [sgmltag]+<r:legend>+, [sgmltag]+<r:xaxis>+, [sgmltag]+<r:yaxis>+ and [sgmltag]+<r:point>+. Each of them customizes a part of chart. All of them are optional except at least one [sgmltag]+<r:series>+ tag is required. It allows to handle fired events using client-side JavaScript or server-side listener.
Copy link
Member

Choose a reason for hiding this comment

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

It allows to handle fired events using client-side JavaScript or server-side listener.

I't not clear what "It" is here. The previous tag you mentioned? or charts altogether?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I meant the chart component in general.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks @Lukindo, I updated the chart-docs branch to more clearly reflect your intention.

@bleathem
Copy link
Member

These docs are a great start. It identifies the relevant tags and provides some "getting started" examples. Further documentation examples can get use-case specific and provide some real-world examples.

@bleathem
Copy link
Member

@lfryc this pull request looks good to merge from my perspective (merge from the chart-docs branch). Do you have any additional comments before we proceed with the merge?

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

Successfully merging this pull request may close these issues.

2 participants