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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Chrome's quota numbers. #3896

Merged
merged 1 commit into from Sep 23, 2020

Conversation

pwnall
Copy link
Contributor

@pwnall pwnall commented Sep 16, 2020

Quota heuristics are a complex area. Sorry for the confusion here! 馃槩

  • Chrome uses up to 80% of total disk space for web storage. Source here.
  • Out of this 80%, Chrome is willing to dedicate at most 75% to a single origin. Source here. 75% * 80% = 60% total disk space / origin.

@pwnall pwnall requested a review from a team as a code owner Sep 16, 2020
@pwnall pwnall requested review from kaycebasques and jpmedley Sep 16, 2020
@googlebot googlebot added the cla: yes Contributor has signed the CLA label Sep 16, 2020
@github-actions
Copy link

github-actions bot commented Sep 16, 2020

Alex Recommends Report

Alex recommends the following language changes, but Alex is a regular expression based algorithm, so take them with a grain of salt.

src/site/content/en/blog/storage-for-the-web/index.md

Level Location Word Recommendation
鈿狅笍 43:4 easy easy may be insensitive, try not to use it
鈿狅笍 74:1 easy easy may be insensitive, try not to use it
鈿狅笍 156:30 easily easily may be insensitive, try not to use it
鈿狅笍 160:43 simple simple may be insensitive, try not to use it
鈿狅笍 161:70 easy easy may be insensitive, try not to use it
鈿狅笍 252:47 simple simple may be insensitive, try not to use it
鈿狅笍 274:1 Special Special may be insensitive, use Has a disability, Person with a disability, People with disabilities instead
鈿狅笍 283:5 hero hero may be insensitive, use role-model, mentor instead

@netlify
Copy link

netlify bot commented Sep 16, 2020

Deploy preview for web-dev-staging ready!

Built with commit e3c698f

https://deploy-preview-3896--web-dev-staging.netlify.app

@jpmedley
Copy link
Contributor

jpmedley commented Sep 16, 2020

@petele The text looks good. If there's nothing you want to add, I'll approve and merge.

@jpmedley jpmedley self-assigned this Sep 16, 2020
@kaycebasques
Copy link
Contributor

kaycebasques commented Sep 23, 2020

Ping @petele

petele
petele approved these changes Sep 23, 2020
@kaycebasques kaycebasques merged commit 50701ec into GoogleChrome:master Sep 23, 2020
1 check passed
@pwnall
Copy link
Contributor Author

pwnall commented Sep 24, 2020

Thank you very much for the reviews!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Contributor has signed the CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants