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

Suggestion: append a "Read-only" label in the editor tabs for files that are read only #53022

Closed
miguelsolorio opened this issue Jun 26, 2018 · 10 comments
Assignees
Labels
feature-request Request for new features or functionality release-notes Release notes issues ux User experience issues verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@miguelsolorio
Copy link
Contributor

Issue Type: Bug

Testing #52573

When you have multiple workspaces (including a read-only one), it's hard to tell which files opened are the read-only ones. Wondering if it's worth appending a Read-only label so it's easier to see when you're in other tabs.

image

VS Code version: Code - Insiders 1.25.0-insider (923e9cc, 2018-06-26T13:00:22.932Z)
OS version: Darwin x64 17.6.0

System Info
Item Value
CPUs Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)
GPU Status 2d_canvas: enabled
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: enabled
rasterization: enabled
video_decode: enabled
video_encode: enabled
vpx_decode: enabled
webgl: enabled
webgl2: enabled
Load (avg) 2, 2, 2
Memory (System) 16.00GB (0.38GB free)
Process Argv /Applications/Visual Studio Code - Insiders.app/Contents/MacOS/Electron -psn_0_4183037
Screen Reader no
VM 0%
Extensions (4)
Extension Author (truncated) Version
svgo 100 0.8.0
vscode-svgo ajh 0.1.0
gitlens eam 8.4.1
python ms- 2018.6.0

(4 theme extensions excluded)

@isidorn
Copy link
Contributor

isidorn commented Jun 27, 2018

I like the idea of specially decorating the tab - not sure if add read-only is the best due to limitied horizontal space.
@bpasero thoughts?

@isidorn isidorn added this to the On Deck milestone Jun 27, 2018
@isidorn isidorn added the feature-request Request for new features or functionality label Jun 27, 2018
@bpasero
Copy link
Member

bpasero commented Jun 27, 2018

@isidorn @misolori we can do it, a similar thing exists when a file is deleted on disk:

image

There might be a better way to indicate these things, so maybe worthwhile for a UX discussion.

@isidorn
Copy link
Contributor

isidorn commented Jun 27, 2018

Yeah, I can bring it up in next weeks ux call

@isidorn isidorn modified the milestones: On Deck, July 2018 Jun 27, 2018
@isidorn isidorn added the ux User experience issues label Jun 27, 2018
@isidorn isidorn closed this as completed in bbd1d81 Aug 2, 2018
@isidorn
Copy link
Contributor

isidorn commented Aug 2, 2018

I like being consistedf with deleted from disk, so I went with that approach.

@roblourens
Copy link
Member

Hey @isidorn I'm seeing this show up at the wrong time

image

isidorn added a commit that referenced this issue Aug 3, 2018
@isidorn
Copy link
Contributor

isidorn commented Aug 3, 2018

Thanks @roblourens for catching that, I did a n00b mistake, pushed a fix. Is it ok now?

@isidorn isidorn added the verification-needed Verification of issue is requested label Aug 3, 2018
@vvs
Copy link

vvs commented Aug 3, 2018

In the latest insiders build, basically all my tabs are now with (read-only). :)

@isidorn
Copy link
Contributor

isidorn commented Aug 3, 2018

@vvs yeah I was very stupid. There will be a new insiders release soon, you will survive for a couple of hours :)

@vvs
Copy link

vvs commented Aug 3, 2018

Thank you, @isidorn, I barely survived, but the latest insiders build fixed the problem! :)

But seriously, many, many thanks for all the hard work you guys are putting in the amazingVS Code, which makes my life much, much easier! :)

@isidorn
Copy link
Contributor

isidorn commented Aug 3, 2018

@vvs glad to hear, you are welcome. Enjoy :)

@weinand weinand added the verified Verification succeeded label Aug 6, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality release-notes Release notes issues ux User experience issues verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants