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

Confusing (?) navigation #3467

Closed
stevemessick opened this issue Jun 8, 2012 · 2 comments
Closed

Confusing (?) navigation #3467

stevemessick opened this issue Jun 8, 2012 · 2 comments
Assignees
Labels
closed-as-intended Closed as the reported issue is expected behavior
Milestone

Comments

@stevemessick
Copy link
Contributor

In the code:
  Event event = new Event('TextEvent');
Selecting the first occurance of "Event" and hitting "Open Declaration" yields the Event Interface.
Selecting the second "Event" and hitting "Open Declaration" yields the factory constructor.
This behavior is confusing.
////////////////////////////////////////////////////////////////////////////////////
Editor Version: 8124

messick writes:
I disagree that this is confusing. It is working as intended. If there's a problem it is in the language definition that allows unnamed constructors, but I don't think that is a problem, either.

@stevemessick
Copy link
Contributor Author

Added AsDesigned label.

@stevemessick
Copy link
Contributor Author

Removed this from the Later milestone.
Added this to the M1 milestone.

@stevemessick stevemessick added Type-Defect closed-as-intended Closed as the reported issue is expected behavior labels Aug 28, 2012
@stevemessick stevemessick self-assigned this Aug 28, 2012
@stevemessick stevemessick added this to the M1 milestone Aug 28, 2012
copybara-service bot pushed a commit that referenced this issue Jun 17, 2022
Changes:
```
> git log --format="%C(auto) %h %s" c4e9ddc..9bf4289
 https://dart.googlesource.com/pub.git/+/9bf4289d Top-level --colors flag (#3467)
 https://dart.googlesource.com/pub.git/+/54eb1851 Hide --trace in the embedding (#3466)
 https://dart.googlesource.com/pub.git/+/b9751fdb Fix stack-trace expectation (#3460)

```

Diff: https://dart.googlesource.com/pub.git/+/c4e9ddc888c3aa89ef4462f0c4298929191e32b9~..9bf4289d6fd5d6872a8929d6312bbd7098f3ea9c/
Change-Id: I5313721779a5a2cda825316a8ece6ed0cd57260e
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/248803
Reviewed-by: Jonas Jensen <[email protected]>
Auto-Submit: Sigurd Meldgaard <[email protected]>
Reviewed-by: Sarah Zakarias <[email protected]>
Commit-Queue: Sigurd Meldgaard <[email protected]>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-as-intended Closed as the reported issue is expected behavior
Projects
None yet
Development

No branches or pull requests

1 participant