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

Location override using "Sensors" pane in the Chrome/Edge dev tools #32463

Closed
guardrex opened this issue Apr 30, 2024 · 3 comments · Fixed by #32464
Closed

Location override using "Sensors" pane in the Chrome/Edge dev tools #32463

guardrex opened this issue Apr 30, 2024 · 3 comments · Fixed by #32464

Comments

@guardrex
Copy link
Collaborator

Description

Per dotnet/aspnetcore#53707 (comment).

Hisham ... I'll float something to address this and ping you on the PR. The easy thing to do would be to cross-link to the product unit issue remark in the last section, and perhaps I'll do that. However, I might write something on this.

BTW -- I'd also like to touch that existing PU issue cross-link. We should have link text, and we have a link text format for PU cross-links. I'll drop that into the PR as well.

Also ... This problem that the dev faced came up in the context of a Blazor app. Therefore, I'll need to address it in the Blazor glob/loc article, too. I'll get that on the PR.

Page URL

https://learn.microsoft.com/en-us/aspnet/core/fundamentals/troubleshoot-aspnet-core-localization?view=aspnetcore-8.0

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/troubleshoot-aspnet-core-localization.md

Document ID

9a1dccda-d7bf-1eb5-e5e3-8aba4aeb444f

Article author

@hishamco

@guardrex guardrex self-assigned this Apr 30, 2024
@dotnet dotnet deleted a comment from github-actions bot Apr 30, 2024
@guardrex guardrex added Pri2 Priority 2 doc-enhancement and removed Source - Docs.ms Docs Customer feedback via GitHub Issue labels Apr 30, 2024
@hishamco
Copy link
Member

I will look at the linked comment, so I can understand how it's related to the troubleshooting article that I wrote long time ago

@guardrex
Copy link
Collaborator Author

Ok ... Halter floated the idea that it would go here. I guessed the same.

The article could use some NIT touches, mostly per our style manual. Even if we don't add the sensors bit to this article, I'll perform a quick edit.

@guardrex
Copy link
Collaborator Author

BTW ... If you'd rather work on this issue, that's totally cool with me. However, let me get a PR merged first for this article with these updates. I've already started working on it, and it will be up shortly. After that merges, you can take over and work this issue if you want to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants