Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Indicate mutable members in editor. #37517

Closed
JohanLarsson opened this issue Jul 26, 2019 · 5 comments · Fixed by #51889
Closed

Indicate mutable members in editor. #37517

JohanLarsson opened this issue Jul 26, 2019 · 5 comments · Fixed by #51889
Assignees
Labels
4 - In Review A fix for the issue is submitted for review. Area-IDE Feature Request
Milestone

Comments

@JohanLarsson
Copy link

Not sure how to render it best but it is useful info. I'm thinking maybe make the last char of the identifier of a mutable member in some distinct different colour. That may not work well for single char local variable names though. FSharp has had this for a while.

@CyrusNajmabadi
Copy link
Member

@JohanLarsson can you show the picture from f#?

@JohanLarsson
Copy link
Author

@ReedCopsey can you help us out with a screenie?

@JohanLarsson
Copy link
Author

An alternative could be to animate opacity/visibility for mutables so that they flicker like broken fluorescent lamps :)

@jinujoseph jinujoseph added Area-IDE Feature Request Need Design Review The end user experience design needs to be reviewed and approved. labels Jul 30, 2019
@jinujoseph jinujoseph added this to the Backlog milestone Jul 30, 2019
@jnm2
Copy link
Contributor

jnm2 commented Oct 30, 2019

I would like an option that changes the look of usages at or after a possible reassignment has occurred, but not before that point.

@jnm2
Copy link
Contributor

jnm2 commented Oct 30, 2019

Screenshot of Roslyn code that I stole from @alrz, demonstrating a ReSharper style option:

image

(What I'd prefer is no underline in if (workingDirectory == null) but underlines everywhere below it in the file.)

@sharwell sharwell added 4 - In Review A fix for the issue is submitted for review. and removed Need Design Review The end user experience design needs to be reviewed and approved. labels Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - In Review A fix for the issue is submitted for review. Area-IDE Feature Request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants