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

add docs for noLegacyStyle flag #238

Merged
merged 1 commit into from Jun 11, 2013
Merged

Conversation

anssiko
Copy link
Member

@anssiko anssiko commented Jun 11, 2013

@marcoscaceres thanks for reminding me of missing docs. This should address #237.

@marcoscaceres
Copy link
Member

@anssiko np. It appears the build is failing?

@@ -475,6 +475,11 @@
If this document is of a status that usually goes with Rec Track, but isn't intended for it,
set this to true.
</dd>
<dt>noLegacyStyle</dt>
<dd>
if set, removes legacy DOM-style sections (Attributes etc.) following WebIDL blocks. This
Copy link
Member

Choose a reason for hiding this comment

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

You should probably say that it's a boolean?

darobin added a commit that referenced this pull request Jun 11, 2013
@darobin darobin merged commit c96be76 into w3c:develop Jun 11, 2013
@darobin
Copy link
Member

darobin commented Jun 11, 2013

Thanks for the docs! I reckon people can figure out it's boolean.

This isn't what was failing the build (Travis is weird sometimes).

@marcoscaceres
Copy link
Member

I've added a note in the wiki to expand this documentation. It's good to give justification as to why not using the legacy style is a good thing.

@anssiko anssiko deleted the noLegacyStyle-docs branch June 12, 2013 09:22
@marcoscaceres
Copy link
Member

It's not clear to me how, once enabled, the markup is supposed to be changed. I can see a lot of content gets removed from a spec once this is enabled. Can you provide some details as to how then a spec is supposed to be edited or point to a spec that was edited using this style?

@marcoscaceres
Copy link
Member

(please reopen this bug :) )

@marcoscaceres
Copy link
Member

FWIW, I'm trying to migrate http://telephony.sysapps.org/ right now, and could really use some guidance.

@anssiko
Copy link
Member Author

anssiko commented Jun 17, 2013

@marcoscaceres, you can look at the Battery Status API as an example:

https://dvcs.w3.org/hg/dap/raw-file/default/battery/Overview.html
https://dvcs.w3.org/hg/dap/raw-file/default/battery/Overview.src.html

Basically you'll just need to move the prose from Attributes and Methods sections and put it below the IDL markup definition. In practice, you may need to adapt the language a bit and link the IDL members back to the prose (ReSpec automatically generates fragment identifiers that start with "#widl-", just add corresponding ids to appropriate parts in the prose and you're done).

Let me know if you run into any issues.

@marcoscaceres
Copy link
Member

Thanks @anssiko! really appreciate the pointers and guidance!

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.

None yet

3 participants