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.

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

Provide UI hints #212

Merged
merged 15 commits into from
Dec 9, 2015
Merged

Provide UI hints #212

merged 15 commits into from
Dec 9, 2015

Conversation

jimCresswell
Copy link
Member

  • Better colour highlighting of interactive elements.
  • Help message tool tips.

@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 8, 2015 13:51 Active
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 8, 2015 14:14 Active
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 8, 2015 15:21 Active
@jimCresswell
Copy link
Member Author

@sponte I bumped the version. Also, I can't reproduce the rendering issue when serving from my machine, can you accept the merge so we can see if the deployed version does this?

@sponte
Copy link
Contributor

sponte commented Dec 9, 2015

Let's try to reproduce on my local today. I don't want to merge potentially unstable code

@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 9, 2015 09:55 Active
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 9, 2015 13:26 Active
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 9, 2015 15:12 Active
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 9, 2015 15:30 Active
…t a catch function, so every single then needs an error handling function...
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 9, 2015 15:52 Active
… clicked, add a page load timeout to the next step because the page has to reload with the new branch.
@jimCresswell jimCresswell deployed to oss-specs-specs-eu-pr-212 December 9, 2015 16:15 Active
@jimCresswell
Copy link
Member Author

@sponte 💥 ❗

sponte added a commit that referenced this pull request Dec 9, 2015
@sponte sponte merged commit a6e8d1a into master Dec 9, 2015
@sponte sponte deleted the feature/uiHints branch December 9, 2015 18:04
@sponte
Copy link
Contributor

sponte commented Dec 9, 2015

@jimCresswell well done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants