-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Audio "Preload" settings copy in the editor are unclear #7726
Comments
This could be a good opportunity to add some help text, since this isn't a very intuitive setting: Edit: btw, the external link would lead to https://wordpress.org/support/article/audio-shortcode/#options. |
One other note might be that you can also opt to not provide a preload setting. If you don't provide the setting, it allows the browser to determine how they want to preload the data. (Chrome uses "metadata" by default, for instance.) Not sure how that might look — a "reset" button or something like that? |
I wonder if "browser default" is worth adding as a fourth, default option in that case? |
While I think the additional text helps, it feels like a lot to read. Here's a pass at trimming it down. We also call it "Preload" and then mention "Load." Does that cause any confusion? Preload
I'm not sure it helps, but just thinking through options. |
I don't love the preload/load thing; the preload seems a little pointless here, since functionally we're using preload/load to mean the same thing here and it's not like there are other load-specific settings that need to be distinguished. If we're going to use both terms, they should mean two different things. For me, a relatively non-technical person, what would be more helpful than text explaining each choice would be text explaining why I should care about this options / why I'd make one choice over another -- all the help text in the world doesn't actually help me if it doesn't help me decide. That then gives context to the options, so you don't have to include help text for every option. So something like:
|
Thanks @michelleweber, this helps a lot. Here's the revised copy: I still think we need to somehow explain what "metadata" entails. |
You could avoid metadata altogether and call it something like "Title and artist info" -- I know it's not 100% accurate, but I'm not sure it matters? Otherwise, I think explaining adds too much text, and I feel like you should rely on the link to do the explaining for you. |
I wonder if we could use |
It feels like this is moving toward the right direction here—this is such a complicated concept to explain well! Maybe instead of "File Options", we call it "Load Options" or "Preload Options"? Instead of "The less you preload...." we could put load speeds next to each option, so:
|
What would each of those speeds be relative to? "Preload nothing" is fast when loading the page, but slower when trying to load the actual audio file. This could still work, but I feel if we remove that second sentence from the description, it becomes a bit too vague. |
And also, as a reminder, these are only hints for the browser. Depending on your browser and configuration, browsers are allowed to completely ignore this setting and download whatever they want.
|
Let's not overthink this — the goal is to explain what this setting is, and why it matters. We don't need to get super specific, and we can put technical details and additional information in the documentation. |
Closing this for now, if we run into some better suggestions we can always revisit. |
The audio preload settings are unclear because they are exposing implementation details from the
<audio>
tag. Instead of "Auto" and "Metadata", something like "Content" and "Metadata only".See: #7590 (comment)
The text was updated successfully, but these errors were encountered: