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

fix(vouchers): ui-select on small screen #1018

Merged

Conversation

jniles
Copy link
Collaborator

@jniles jniles commented Dec 3, 2016

This commit makes sure that the UI-select on the complex vouchers page
has plenty of room to select accounts via the css class
ui-grid-ui-select applied to the ui-select-choices element.

All other ui-select elements in grids can take advantage of this class
to display themselves better.

Closes #873.

uigriduiselect
Fig 1: The present implementation described.


Thank you for contributing!

Before submitting this pull request, please verify that you have:

  • Run your code through JSHint. Check out our styleguide.
  • Run integration tests.
  • Run end-to-end tests.
  • Accurately described the changes your are making in this pull request.

For a more detailed checklist, see the official review checklist that this PR will be evaluated against.

Ensuring that the above checkboxes are completed will help speed the review process and help build a stronger application. Thanks!

This commit makes sure that the UI-select on the complex vouchers page
has plenty of room to select accounts via the css class
`ui-grid-ui-select` applied to the `ui-matches` element.

All other `ui-select` elements in grids can take advantage of this class
to display themselves better.

Closes IMA-WorldHealth#873.
@jniles jniles added the Bug Fix label Dec 3, 2016
@jniles jniles added this to the Finance Implementation milestone Dec 3, 2016
Copy link
Contributor

@sfount sfount left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This has been needed for a long time. Excellent fix.

@sfount sfount merged commit 46b3bda into IMA-WorldHealth:master Dec 3, 2016
@jniles jniles deleted the fix-voucher-ui-select-css-rules branch December 4, 2016 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants