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

Review and integrate open283 #360

Merged
merged 1 commit into from
Dec 1, 2015
Merged

Review and integrate open283 #360

merged 1 commit into from
Dec 1, 2015

Conversation

charlesh88
Copy link
Contributor

Fixes CSS overflow problem that was hiding toolbar dropdown menus.

open #199
open #283
Tweaks to .tool-bar layout to handle very narrow widths.

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

@charlesh88
[Frontend] Fixed overflow problem that was hiding toolbar menu
5fd5593 
open #199
open #283
Tweaks to .tool-bar layout to handle very
narrow widths;
@charlesh88 charlesh88 added this to the Baxter milestone Dec 1, 2015
@charlesh88 charlesh88 mentioned this pull request Dec 1, 2015
Closed
@akhenry
Copy link
Contributor

akhenry commented Dec 1, 2015

Looks good.

Reviewer Checklist

  1. Changes appear to address issue? Y
  2. Appropriate unit tests included? N/A
  3. Code style and in-line documentation are appropriate? Y
  4. Commit messages meet standards? Y

akhenry added a commit that referenced this pull request Dec 1, 2015
@akhenry
Merge pull request #360 from nasa/open283
4798cb3 
Review and integrate open283
@akhenry akhenry merged commit 4798cb3 into open199 Dec 1, 2015
@akhenry akhenry deleted the open283 branch December 1, 2015 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@akhenry akhenry

Labels
None yet
Projects
None yet
Milestone
Baxter
Development

Successfully merging this pull request may close these issues.
2 participants
@charlesh88 @akhenry