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.

Sign up for GitHub

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

Jump to bottom

Ctrl-Alt-R only works in http://www.dartlang.org/docs/getting-started/ when textarea has input focus #27

Closed
DartBot opened this issue Oct 10, 2011 · 1 comment
Closed

Ctrl-Alt-R only works in http://www.dartlang.org/docs/getting-started/ when textarea has input focus #27

DartBot opened this issue Oct 10, 2011 · 1 comment
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue

Comments

@DartBot
Copy link

DartBot commented Oct 10, 2011

This issue was originally filed by adrian.ai...@gmail.com

What steps will reproduce the problem?

  1. Visit http://www.dartlang.org/docs/getting-started/
  2. Move pointer over the (|>) start button, notice callout "Run (Ctrl-Alt-R)"
  3. Press Ctrl-Alt-R and notice as nothing happens.
  4. Click in the source code textarea and notice how Ctrl-Alt-R now works.

What is the expected output? What do you see instead?

I expected the keyboard shortcut to work as advertized, without having to set focus in the textarea.

If this is not feasible, then the callout should only advertize the shortcut when applicable, i.e. focus is inside the textarea.

What version of the product are you using? On what operating system?

Above URL as of this issue submit time.

Please provide any additional information below.
@DartBot
Copy link
Author

DartBot commented Oct 11, 2011

This comment was originally written by drfibonacci@google.com

Good catch, but not worth fixing as it's most useful when you're in the text box, anyway.

Added Component-Docs, WontFix labels.

@kodandersson kodandersson mentioned this issue Jun 23, 2015
Closed
@iposva-google iposva-google mentioned this issue Sep 21, 2015
Closed
@whesse whesse mentioned this issue Nov 4, 2015
Closed
@zoechi zoechi mentioned this issue Nov 21, 2015
Closed
This was referenced Dec 2, 2015
Closed
Closed
@zoechi zoechi mentioned this issue Jan 7, 2016
Closed
@kzhdev kzhdev mentioned this issue Feb 14, 2016
Closed
@bwilkerson bwilkerson mentioned this issue Feb 19, 2016
Closed
@zoechi zoechi mentioned this issue Feb 29, 2016
Closed
@kevmoo kevmoo added closed-not-planned Closed as we don't intend to take action on the reported issue and removed resolution-wont_fix labels Mar 1, 2016
@whesse whesse mentioned this issue Mar 3, 2016
Closed
@skybrian skybrian mentioned this issue Mar 6, 2016
Closed
@zoechi zoechi mentioned this issue Mar 10, 2016
Closed
This was referenced Mar 10, 2016
Closed
Closed
This was referenced Mar 12, 2016
Closed
Closed
@johnmccutchan johnmccutchan mentioned this issue Apr 7, 2016
Closed
11 tasks
@bwilkerson bwilkerson mentioned this issue May 17, 2016
Closed
@whesse whesse mentioned this issue Jun 30, 2016
Closed
@whesse whesse mentioned this issue Aug 18, 2016
Closed
@johnpryan johnpryan mentioned this issue Sep 14, 2016
Closed
@a-siva a-siva mentioned this issue Sep 27, 2016
Closed
@MuteMuty MuteMuty mentioned this issue Dec 23, 2022
Closed
@Mmisiek Mmisiek mentioned this issue Feb 14, 2023
Closed
@thor-jc thor-jc mentioned this issue Feb 19, 2023
Open
@adityabansalx adityabansalx mentioned this issue Mar 25, 2023
Closed
@AdityaKha AdityaKha mentioned this issue Apr 3, 2023
Closed
@ghost ghost mentioned this issue May 11, 2023
Closed
@Mufaddal1125 Mufaddal1125 mentioned this issue May 13, 2023
Closed
@FarhanAli-98 FarhanAli-98 mentioned this issue Jun 8, 2023
Closed
copybara-service bot pushed a commit that referenced this issue Jul 21, 2023
@jiahaog
[dart2wasm] Fix location RangeError when using part
11d0f06 
Here's a minimal repro that this CL fixes:

`ui.dart`

```dart
library dart.ui;

import 'dart:ffi';

part 'foo.dart';
```

`foo.dart`

```dart
part of dart.ui;

@Native<Void Function()>(symbol: 'foo_func', isLeaf: true)
external void foo_func();
```

When compiling with `compile_platform.dart` with `--target=dart2wasm`, the following error appears:


```
Unhandled exception:
Verification error: Target=wasm, VerificationStage.afterModularTransformations: Invalid location with target 'wasm' on FunctionNode() (FunctionNode): RangeError (offset): Invalid value: Not in inclusive range 0..56: 91
Context: 'foo_func_$import'.
Node: 'FunctionNode()'.
#0      VerificationErrorListener.reportError (package:kernel/verifier.dart:81:5)
#1      VerifyingVisitor.problem (package:kernel/verifier.dart:222:14)
#2      VerifyingVisitor._getLocation (package:kernel/verifier.dart:1361:7)
#3      VerifyingVisitor._hasLocation (package:kernel/verifier.dart:1370:26)
#4      VerifyingVisitor.getSameLibraryLastSeenTreeNode (package:kernel/verifier.dart:1342:28)
#5      VerifyingVisitor.localContext (package:kernel/verifier.dart:1382:24)
#6      VerifyingVisitor.defaultDartType (package:kernel/verifier.dart:1491:41)
#7      Visitor.visitVoidType (package:kernel/visitor.dart:1309:37)
#8      VoidType.accept (package:kernel/ast.dart:11190:42)
#9      FunctionNode.visitChildren (package:kernel/ast.dart:3919:16)
#10     VerifyingVisitor.visitChildren (package:kernel/verifier.dart:259:10)
#11     VerifyingVisitor.visitWithLocalScope (package:kernel/verifier.dart:266:5)
#12     VerifyingVisitor.visitFunctionNode (package:kernel/verifier.dart:721:5)
#13     FunctionNode.accept (package:kernel/ast.dart:3908:38)
#14     VerifyingVisitor.visitProcedure (package:kernel/verifier.dart:620:19)
#15     Procedure.accept (package:kernel/ast.dart:3311:40)
#16     visitList (package:kernel/ast.dart:14488:14)
#17     Library.visitChildren (package:kernel/ast.dart:591:5)
#18     VerifyingVisitor.visitChildren (package:kernel/verifier.dart:259:10)
#19     VerifyingVisitor.defaultTreeNode (package:kernel/verifier.dart:196:5)
#20     TreeVisitor.visitLibrary (package:kernel/visitor.dart:503:35)
#21     VerifyingVisitor.visitLibrary (package:kernel/verifier.dart:367:11)
#22     Library.accept (package:kernel/ast.dart:577:38)
#23     visitList (package:kernel/ast.dart:14488:14)
#24     Component.visitChildren (package:kernel/ast.dart:14320:5)
#25     VerifyingVisitor.visitChildren (package:kernel/verifier.dart:259:10)
#26     VerifyingVisitor.visitComponent (package:kernel/verifier.dart:342:7)
#27     Component.accept (package:kernel/ast.dart:14313:38)
#28     VerifyingVisitor.check (package:kernel/verifier.dart:171:15)
#29     verifyComponent (package:kernel/verifier.dart:69:20)
...
```

The issue seems to be that after doing this native transformation, the node's `fileUri` references the enclosing library (`ui.dart` above), but the `node.location` references the actual source file (`foo.dart` above) indirectly through `node.fileOffset`.

This ends up being an issue when compiling the platform dill in Google3,   but I didn't look into why `flutter build web --wasm` isn't broken.

Internal bug: b/292172146

Change-Id: I2b8d7d215b2c36354860257ce651d50168e9523d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/315360
Reviewed-by: Ömer Ağacan <omersa@google.com>
Commit-Queue: Jia Hao Goh <jiahaog@google.com>
@ebraminio ebraminio mentioned this issue Aug 18, 2023
Closed
@shubham-jain2 shubham-jain2 mentioned this issue Aug 24, 2023
Closed
@llfbandit llfbandit mentioned this issue Sep 10, 2023
Closed
@bernardolansing bernardolansing mentioned this issue Sep 27, 2023
Closed
@manandhar-aarya manandhar-aarya mentioned this issue Oct 9, 2023
Closed
@cosminbodnariuc cosminbodnariuc mentioned this issue Dec 7, 2023
Closed
@mnts mnts mentioned this issue Feb 6, 2024
Open
@JgomesAT JgomesAT mentioned this issue Feb 28, 2024
Closed
@adityasaxen adityasaxen mentioned this issue Mar 14, 2024
Closed
@upasana-khatiwada upasana-khatiwada mentioned this issue Mar 16, 2024
Closed
@butterbrain butterbrain mentioned this issue Apr 4, 2024
Closed
@abhisiroha abhisiroha mentioned this issue Apr 24, 2024
Closed
@eernstg eernstg mentioned this issue Jul 1, 2024
Open
@Lootwig Lootwig mentioned this issue Jul 18, 2024
Open
@the-mrinal the-mrinal mentioned this issue Jul 31, 2024
Open
@ailabs-software ailabs-software mentioned this issue Aug 6, 2024
Closed
@thermoworker thermoworker mentioned this issue Aug 12, 2024
Open
@theaquaflower theaquaflower mentioned this issue Aug 15, 2024
Closed
@SwederSchellens SwederSchellens mentioned this issue Sep 24, 2024
Closed
@ChaserVasya ChaserVasya mentioned this issue Oct 20, 2024
Open
Open
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
closed-not-planned Closed as we don't intend to take action on the reported issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kevmoo @DartBot