zed icon indicating copy to clipboard operation
zed copied to clipboard

completion box: add strikethrough for deprecated methods

Open kzhui125 opened this issue 1 year ago • 1 comments

Check for existing issues

  • [X] Completed

Describe the bug / provide steps to reproduce it

for example:

trait std::ascii::AsciiExt is deprecated. Here is make_ascii_lowercase in VSCode:

image

Environment

Zed: v0.123.3 (Zed) OS: macOS 14.3.1 Memory: 32 GiB Architecture: aarch64

If applicable, add mockups / screenshots to help explain present your vision of the feature

No response

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.

No response

kzhui125 avatar Feb 25 '24 15:02 kzhui125

A PR just landed to add in strikethrough support:

  • https://github.com/zed-industries/zed/pull/8287

So the gut instinct that Kirill and I have is that this shouldn't be too hard to add. :)

JosephTLyons avatar Feb 27 '24 20:02 JosephTLyons

Hey, I think I could fix this (its my first time contributing) should I just open a PR? Thanks!

Adam-Alani avatar Mar 08 '24 22:03 Adam-Alani

Hey, I think I could fix this (its my first time contributing) should I just open a PR? Thanks!

I'd say go for it. You can check the contribution guide for the best way to do this.

Moshyfawn avatar Mar 08 '24 22:03 Moshyfawn

@JosephTLyons Can this issue be closed now or do we have more touch points to complete for this issue? Went through the PR I couldn't get clarity there hence asking.

manuraj17 avatar Mar 12 '24 11:03 manuraj17

This feature has not been released yet.

image

Moshyfawn avatar Mar 12 '24 12:03 Moshyfawn

@Moshyfawn Oh okay 👍🏽 I was under the impression issue gets closed on PR merge.

manuraj17 avatar Mar 12 '24 13:03 manuraj17

@Moshyfawn Oh okay 👍🏽 I was under the impression issue gets closed on PR merge.

I see! I was under the impression that you were looking to see if the feature had been implemented yet 🙈 Good catch!

Moshyfawn avatar Mar 12 '24 13:03 Moshyfawn

Implemented in #9086 and coming soon to Peview.

Moshyfawn avatar Mar 12 '24 13:03 Moshyfawn

not working in some cases:

@Moshyfawn @Adam-Alani

Thanks,

[dependencies]
chrono = "0.4"
NaiveDate::from_y
image image

kzhui125 avatar Apr 23 '24 06:04 kzhui125