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

Issue/936 - Use WordPress .nav-tab-wrapper class in Donor screen tabs #938

Merged
merged 3 commits into from
Aug 23, 2016

Conversation

ramiy
Copy link
Contributor

@ramiy ramiy commented Aug 22, 2016

Resolving #936

@ramiy
Copy link
Contributor Author

ramiy commented Aug 22, 2016

Not ready yet, needs more CSS love to align the icons

@DevinWalker
Copy link
Member

I can take care of the CSS

@DevinWalker DevinWalker merged commit cfcceb6 into impress-org:release/1.7 Aug 23, 2016
@ramiy ramiy deleted the issue/936 branch August 23, 2016 17:08
@ramiy ramiy restored the issue/936 branch August 23, 2016 23:38
@ramiy ramiy deleted the issue/936 branch August 23, 2016 23:58
DevinWalker pushed a commit that referenced this pull request Aug 24, 2016
* release/1.7: (22 commits)
  Space remove
  Fixed incorrect variable name
  removed unnecessary var
  CSS improvements for donors section due to merge of #938 re: #936
  Sentence closures and doc block improvements
  Do not format amount which is already formatted
  Added a readme item for #780
  Added back the gateway column #780
  Removed the gateway column #780
  Improved layout of error logs page, doc block updates
  Change log item for #937 added
  Ensure API and System Info tabs don't have save button output #937
  Reverted tabs to pre-1.5 state - a page reload after clicking each tab #937
  a11y: add `<label>` fields to newsletter `<input>` fields, with `for` attributes and `class="screen-reader-text"`
  Remove Give Tabs (`#customer-tab-wrapper-list`), replaced by WordPress Tabs (`.nav-tab-wrapper`)
  Use WordPress `.nav-tab-wrapper` class in Donor screen tabs
  Doc block updates
  Auto format amount with decimal separator when focusout from money field
  Fix currency icon before input field css bug
  Remove all decimal seperator except last if thousand seperator is same ad decimal
  ...
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