-
-
Notifications
You must be signed in to change notification settings - Fork 304
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
Re write sensors page as text rather than table #167
Comments
You bring up a good point on this. In fact some of the sensors already exist as a sub-section so we may just need to flesh them out a bit more and remove from the table. |
Hi, |
Hi @jonasCr. Yep this is still very much open and help would be hugely appreciated 😄 . Currently the table has most sensors just listed in a table with the description also in the table. This really doesn't work in mobile browsers (see above). Instead I think we want remove the tables and translate the information in to paragraph form. Trying to combine the Android and iOS information where they are common and also incorporate the additional information from the bottom of the page. Something like (images may look poor on GitHub dark mode as we don't have the CSS we use on the actual site) also note I've fixed the previously incorrect entity ID. Activity SensorEntity: Additional Details
If iOS is unable to calculate an activity from motion data, It is possible for multiple activities to be returned, such as The
This sensor is only available on the full flavor of the Android app that is found in the Google Play Store, it is not available for the minimal flavor. For android the user will have a different set of states to go by:
The attribute for the state will reflect the |
Thanks for your answer. I think I will have a few times those days to get into this. Tell me if I'm wrong but the main idea is:
Am I going in the right direction ? |
Yep that sounds pretty much right. Thanks again :) |
The sensors doc looks bad on mobile devices (see below) since the table is too wide for a screen. We should re write this plain text with each sensor as a sub-section heading to allow for easy direct linking. I'm thinking something like how configuration variables are shown in the main docs, see here for example.
Current page in Safari on a iPhone Xs:
And in Chrome on a Galaxy S10:
The text was updated successfully, but these errors were encountered: