Line breaks with $ operators

[gadenbuie]

Niche, but with R6 classes method chaining is common. We have the following line in bslib

tq$find(".accordion-button")$removeClass("collapsed")$removeAttrs("aria-expanded")$addAttrs("aria-expanded" = "true")

that air (0.1.2) reformats as

tq$find(".accordion-button")$removeClass("collapsed")$removeAttrs(
  "aria-expanded"
)$addAttrs("aria-expanded" = "true")

I personally would have rewritten this as

tq$
  find(".accordion-button")$
  removeClass("collapsed")$
  removeAttrs("aria-expanded")$
  addAttrs("aria-expanded" = "true")

Maybe the line break at the first $ could be magical, since most people would probably want to opt into this style.

[lionel-]

Agreed, we should support anything that can be chained in the same way for consistency and usability.

[DavisVaughan]

I don't think this will work exactly the same as binary expressions though, because you probably want a_data_frame$a_really_long_name to always be "sticky" with no newlines on the RHS even if the expression exceeds the line length. So we will have to find something that balances that with the ability to chain.

[lionel-]

you probably want a_data_frame$a_really_long_name to always be "sticky" with no newlines on the RHS

This idea is consistent with #183 (comment)