]> git.lizzy.rs Git - rust.git/commit - src/tools/rust-analyzer
Rollup merge of #87178 - moxian:rd-use, r=jyn514
authorMara Bos <m-ou.se@m-ou.se>
Mon, 16 Aug 2021 21:37:25 +0000 (23:37 +0200)
committerGitHub <noreply@github.com>
Mon, 16 Aug 2021 21:37:25 +0000 (23:37 +0200)
commit2dbb2f7420e72c848d71dcdf61e2a1117d97cc5d
treebac9aae1e926e47b3839bca2ed921e23afdb368a
parent0035d9dcecee49d1f7349932bfa52c05a6f83641
parent81d792f31d02ac461f5ea3f9f65f14cfc95a0e6f
Rollup merge of #87178 - moxian:rd-use, r=jyn514

[rustdoc] Copy only item path to clipboard rather than full `use` statement.

The (somewhat) recent addition of the "copy item import to clipboard" button is extremely nice.

However, i tend to write my code with fully qualified paths wherever feasible and only resort to `use` statements as a refactoring pass. This makes the "copy to clipboard" workflow awkward to use, as i would be copy-pasting that as, say

```rust
impl use std::ops::Add; for MyType {
```

and then go back  and remove the `use ` and `;`.

This PR removes the `use ;` decorations, making it much nicer to use for fully-qualified items. I argue, however, that this does not noticeably degrade experience for those who prefer to import items, since the hard part about those is getting the path right, and writing the `use ;` decoration can be done by hand with little effort.
src/librustdoc/html/render/print_item.rs
src/librustdoc/html/static/js/main.js