[camera] Update README#4988
Conversation
|
@blasten @ditman @jmagman What do you think of the OS support chart here? This is a proposal for what to roll out across all of our plugins as part of flutter/flutter#34999 and flutter/flutter#84200 so if we want a different format we should decide now if possible. (An alternative I considered was to make it a vertical table so there's more room for inline notes, but having the table compact and linking out for notes seemed like the better option.) |
|
Here's what the table looks like:
I think the compact table and asterisk for notes that you have now makes sense. LGTM |
There was a problem hiding this comment.
I like this, thanks for doing it!
In general all the _web plugins (should) have a README with what's supported/not supported by the web with regards to the plugin (or by each browser in particular), so linking to those docs is a good idea.
(I wish we could open the link in a new tab, but users can always center-click to not lose track of what they were reading so far.)
Also: are we going to have all supported platforms as columns? That might be a lot of columns? :)
That's why I'm trying to keep them very short. I considered it as rows, but having potentially 6 rows seemed much more disruptive to the flow of the README. |

Fixes flutter/flutter#83722
Part of flutter/flutter#84200 and flutter/flutter#34999
Pre-launch Checklist
dart format.)[shared_preferences]pubspec.yamlwith an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.CHANGELOG.mdto add a description of the change, following repository CHANGELOG style.///).