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.

Sign up for GitHub

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

Jump to bottom

Tilt sensor show phrase #30

Open
cpseager opened this issue Nov 12, 2015 · 2 comments
Open

Tilt sensor show phrase #30

cpseager opened this issue Nov 12, 2015 · 2 comments
Assignees
@tarling
Labels
Pending review

Comments

@cpseager
Copy link
Collaborator

It would be good now to show the phrase 'this way' etc. on the feedback panel

capturetilt2
@cpseager
Copy link
Collaborator Author

Fixed by f959080 but formatting needs tweaking slightly to fit the longer phrase

tarling added a commit that referenced this issue Nov 16, 2015
@tarling
#30. tweaking display table. bug fix. optimisation
3de69dc
@tarling tarling assigned cpseager and unassigned tarling Nov 16, 2015
@cpseager cpseager assigned tarling and unassigned cpseager Nov 16, 2015
@cpseager
Copy link
Collaborator Author

I've realised this amendment breaks the tilt operation as we are now comparing a number against a string (in 'value').

What we need to do is leave 'value' as numeric, but change the html string shown to the phrase. If we leave number numeric it can also be compared with variables etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tarling tarling

Labels
Pending review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tarling @cpseager