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

Switch to hard tabs for indentation #5260

Closed
Cherry opened this issue Jul 24, 2022 · 1 comment
Closed

Switch to hard tabs for indentation #5260

Cherry opened this issue Jul 24, 2022 · 1 comment
Assignees
Labels
docs engine Requires a change to the docs engine engineering Problems or updates to developers.cloudflare.com website

Comments

@Cherry
Copy link
Contributor

Cherry commented Jul 24, 2022

Which Cloudflare product(s) does this pertain to?

All

Subject Matter

Switch to hard tabs for indentation

Content Location

https://developers.cloudflare.com/*

Additional information

(I couldn't find a more appropriate issue form type, so apologies if this is not accurate)

Continuing from cloudflare/workers-sdk#1298 and cloudflare/templates#76, it would be great to update this repo to also use hard tabs for indentation, both in code examples and any other TS/JS.

With the merge of #5013 recently, there wouldn't be any visual change to how code samples are displayed, but when copied out to editors, would include tabs for better default accessibility.

I would very much encourage folks to read through cloudflare/workers-sdk#1298 for all of the reasons why switching to hard tabs for indentation on this repo would be a good thing.

Given the complexity and fast moving nature of this repo, I didn't want to simply submit a PR for this myself (since the diff would be massive), but am happy to do so if requested.

@Cherry Cherry added content:new Request for new/missing content documentation Documentation edits labels Jul 24, 2022
@deadlypants1973 deadlypants1973 added engineering Problems or updates to developers.cloudflare.com website docs engine Requires a change to the docs engine and removed documentation Documentation edits content:new Request for new/missing content labels Aug 2, 2022
@deadlypants1973 deadlypants1973 removed their assignment Aug 2, 2022
@kodster28
Copy link
Contributor

Closing this out b/c it's tracked in the other associated issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs engine Requires a change to the docs engine engineering Problems or updates to developers.cloudflare.com website
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants