Skip to content

fix #109733 prevent double '(read-only)' suffix in titlebar#109736

Closed
gjsjohnmurray wants to merge 1 commit intomicrosoft:masterfrom
gjsjohnmurray:fix-109733
Closed

fix #109733 prevent double '(read-only)' suffix in titlebar#109736
gjsjohnmurray wants to merge 1 commit intomicrosoft:masterfrom
gjsjohnmurray:fix-109733

Conversation

@gjsjohnmurray
Copy link
Contributor

This PR fixes #109733

@bpasero please review and merge if satisfactory.

@bpasero bpasero added this to the November 2020 milestone Oct 30, 2020
@bpasero
Copy link
Member

bpasero commented Oct 30, 2020

Thanks, I was looking into the code myself for 5 minutes and was not able to think about a solution that is straight forward. I am not a big fan of the approach to introduce a flag, but I will comment more next milestone.

@bpasero
Copy link
Member

bpasero commented Oct 31, 2020

My suggested fix: cfb74bd

@bpasero
Copy link
Member

bpasero commented Nov 2, 2020

Thanks, but I went with the alternate approach.

@bpasero bpasero closed this Nov 2, 2020
@gjsjohnmurray
Copy link
Contributor Author

No problem. Thanks for fixing it a better way.

@gjsjohnmurray gjsjohnmurray deleted the fix-109733 branch November 2, 2020 11:55
@github-actions github-actions bot locked and limited conversation to collaborators Dec 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[regression] Files served by readonly filesystem show double '(read-only)' suffix in titlebar again

2 participants