-
Notifications
You must be signed in to change notification settings - Fork 118
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
Self-referential links on dartdoc pages are not obvious, broken links for inherited methods #1239
Comments
@kwalrath - I'm not sure how those docs are generated, and if this is a dartdoc regression or not. |
@chalin, what's your take on this? |
|
Re (1): It seems silly to have a page link to itself. In this case, couldn't dartdoc detect the self-link and do something smarter? |
Well, that could be open to debate. It does offer visual uniformity for all references. I'm more concerned about situations where I see a type and that it is not linked. That is the case, for example, with dart:core types -- i.e., they are not being linked. As a reader I'd want to follow a link to, e.g., the Map type sometimes. |
This is related (and may be a duplicate depending on how you look at it) to #721. |
I'd say that this issue is distinct enough from #721, in that it refers to in-page links rather than side-nav links to the current page. |
Part one of ghost's comment is working as intended from dartdoc's perspective (if you don't want a self-referential link in your dartdoc, don't write one in the comment). Part two is a real problem with enums that is fixed in #1390. |
Since the first part is WAI and #1390 addressed the second part, closing. |
(1) Links provided in https://angular.io/docs/dart/latest/api/angular2.platform.browser/TemplateSecurityContext-class.html are redirecting to it self.Try clicking on TemplateSecurityContext .It is no point of redirecting to the page it self by providing a link.
(2) links provided on page such as toString and noSuchMethod ( https://angular.io/docs/dart/latest/api/angular2.platform.browser/TemplateSecurityContext-class.html ) giving 404 not found.
The text was updated successfully, but these errors were encountered: