-
Notifications
You must be signed in to change notification settings - Fork 55
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
Consider renaming boxes to bins or something #51
Comments
Or at least maybe just not exporting the definition if it's a spec-internal term... |
@inexorabletash @shuangMoz @jakearchibald thoughts? I'm happy to rename this. We'll need to export something since eventually other specifications will use it. Bin doesn't seem appropriate since it's not trash. I could prefix it with storage perhaps. So "storage box". Other alternative: store. But that is also used a lot I suspect. |
FWIW the "trash" association with "bin" is probably more of a regional thing; it isn't in the dictionary. (But I can believe that some places in the world use "trash bin" like Americans use "garbage can".) |
I'm fine with a rename, but no strong preferences. "store" will get confused with Indexed DB's object store Internally, Chrome has "partition" for a different concept so I'd vote against that one too. I'd casually use "bucket" to describe this, but there may be connotations (buckets get full, buckets leak, etc) |
I'm okay with "bucket". |
We have trash bin and recycling bin here in the US, and they're understandably the most common, but we also have other kinds of bins. :) The plastic things in here are all called bins. https://www.northerntool.com/images/product/2000x2000/502/50273_2000x2000.jpg (The only other term I can think of atm would be “compartment” but it's less precise for this particular usage.) |
This avoids needlessly conflicting with the CSS box model. Fixes #51.
I created #52 to address this. |
Per feedback in the PR it seems like it'll be "bucket" after all. |
This avoids needlessly conflicting with the CSS box model. Fixes #51.
This is a very trivial issue, and maybe there's a very good reason not to do this, but given that “box” in CSS has a very specific definition that's used in pretty much every CSS spec and is embedded as a term in some of its APIs, one might consider renaming the Storage concept of “box” to something else?
The text was updated successfully, but these errors were encountered: