Skip to content
This repository was archived by the owner on Feb 22, 2018. It is now read-only.

docs(formatter): improvements to currency, date, filter, and formatter #1008

Closed
wants to merge 1 commit into from

Conversation

naomiblack
Copy link
Contributor

No description provided.

@naomiblack
Copy link
Contributor Author

I'm not done with filter, but it's a start. @kwalrath (or anyone else) if you have any comments on doc style, I'd love to hear them. I'm trying to make our overall style more consistent with dartdoc.

*
* - `expression` (required) - one of [Map], [Function], [String], [bool], [num]
* - `comparator` (optional)
* <div ng-repeat="item in items | filter:_expression_[:_comparator_]">
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'm using underscore here to reflect things that are user specified. Does that look right to you?

Copy link
Contributor

Choose a reason for hiding this comment

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

To me the explanation below should be enough. Is underscore used somewhere else ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, underscore is something we have used in passed.

*
* [leading]: Symbol should be placed in front of the number
* - `value`: the value to format as currency.
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity: why [] has in been in favor of `` ?

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe that [] creates a link where as `` only changes the font to monospace.

Copy link
Contributor

Choose a reason for hiding this comment

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

And which one is better ?
(I'm asking because the Dart source files use "[]" to document method args)

Copy link
Contributor

Choose a reason for hiding this comment

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

I'd say it's a matter of taste. I personally find unnecessary links annoying.

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'm slowly going through and replacing [ ] that just point back to the same dartdoc page with ``. I think it's confusing to the reader to click something link-like and then realise it only took them a few lines up to the header, and didn't provide any new information.

naomiblack added a commit that referenced this pull request May 7, 2014
@naomiblack naomiblack closed this in db04ce3 May 7, 2014
jbdeboer pushed a commit to jbdeboer/angular.dart that referenced this pull request May 13, 2014
dsalsbury pushed a commit to dsalsbury/angular.dart that referenced this pull request Jul 16, 2014
dsalsbury pushed a commit to dsalsbury/angular.dart that referenced this pull request Jul 16, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

4 participants