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

Review and integrate fix for hover bubble problem #75 #76

Merged
merged 2 commits into from
Aug 24, 2015
Merged

Conversation

charlesh88
Copy link
Contributor

#75

CSS modified to utilize word-break property;

Author checklist:

  • Changes address original issue? Y
  • Unit tests included and/or updated with changes? N/A
  • Command line build passes? N/A
  • Expect to pass code review? Y

#75
CSS modified to utilize word-break property;
#75
Merged in master branch;
Re-rendered platform/.../theme-espresso.css;
@charlesh88
Copy link
Contributor Author

Merge conflict resolved and updated commit pushed.

@larkin
Copy link
Contributor

larkin commented Aug 24, 2015

Review Checklist

Changes appear to address issue? Y
Appropriate unit tests included? N/A (css only)
Code style and in-line documentation are appropriate? Y
Commit messages meet standards? Y
Project-specific information isolated to appropriate branches? Y

Integration Checklist

Automated build passing after merge? Y
No merge conflicts (or resolution trivial/obvious)? Y
All master branches up-to-date after merge? Y
Project-specific information isolated to appropriate branches? Y

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