Bob Nystrom

I really don't know what to say here. Literally the first sentence in the proposal for the new style (#1253) is: > The main change is that you no longer...

I think we have wandered off the path into philosophy. Earlier, I asked: > Today, when you decide to insert a trailing comma to manually force an argument list to...

> @munificent I already answered. See [#1539 (comment)](https://github.com/dart-lang/dart_style/issues/1539#issuecomment-2298166019) (the last paragraph). OK, I'm working on #1528 and related issues now. If those are fixed, does that change how you feel...

I'm going to go ahead and close this because I don't see anything concrete here that I can take action on. The overall proposal to automate adding and remove trailing...

I agree that the output here doesn't look great. That's often the case when you have a split call chain and the target (here just `js`) is shorter than 4...

@Reprevise, your issue is a little different. It should be improved by the #1687 PR I recently landed. With that, your example gets formatted like: ```dart Builder( builder: (context) =>...

Oh, heh. Oops, you're right in this case there is no difference. I thought it was running into #1466 where the formatter would give you: ```dart Builder( builder: (context) =>...

> I'm really not sure why we needed to change the line breaks and indentation of `.map().toList()` chain calls, it was perfect before. The new tall style is a rewrite...

Yes, the first two examples of yours and the last one are because of `await`. I filed #1531 for that. Redirecting factory constructors are an interesting one. I think people...

Hi, @rrousselGit. With #1687 done, I went through the examples. Here is what the formatter produces now: ```dart // 1: final character = await repository.fetchCharacter( id, cancelToken: cancelToken, ); //...