Flutter-intellij: Import quick-fix doesn't obey import order

[goderbauer]

I used the import quickfix Import library 'package:flutter/services.dart', which resulted in the following analyzer warning:

info: Place 'package:' imports before relative imports. (directives_ordering at [flutter] lib/src/widgets/gesture_detector.dart:11)

Import section after quick-fix:

import 'package:flutter/foundation.dart';
import 'package:flutter/gestures.dart';
import 'package:flutter/rendering.dart';

import 'basic.dart';
import 'framework.dart';
import 'package:flutter/services.dart';

Would be nice if the import quickfix would add the import at the right position to not trigger an analyzer warning.

(not super-high priority, but nice-to-have )

[pq]

@scheglov : I thought we did this already... Or?

[scheglov]

Yes, I'm pretty sure we already put newly imported libraries at correct positions.
If it does not work, I would need maybe a bit more specific test case.

[pq]

Hmmmm. Confirmed that this does NOT appear to be working as expected.

Perhaps a regression somewhere?

cc @alexander-doroshko @jwren

[pq]

@scheglov : if the gif doesn't provide enough context, I can demo in person...

[scheglov]

OK, thanks, I can reproduce the problem.

[scheglov]

https://codereview.chromium.org/2955913002

[pq]

Thank you!

[scheglov]

https://github.com/dart-lang/sdk/commit/a2f18fdb7324b3d49c9d43023bd67ab148aa18d6