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

Add frost-fixed-table component #2

Merged
merged 5 commits into from
Dec 15, 2016
Merged

Add frost-fixed-table component #2

merged 5 commits into from
Dec 15, 2016

Conversation

job13er
Copy link
Contributor

@job13er job13er commented Dec 14, 2016

This project uses semver, please check the scope of this pr:

  • #patch# - backwards-compatible bug fix
  • #minor# - adding functionality in a backwards-compatible manner
  • #major# - incompatible API change

NOTE: I'll be adding support for custom cell renderers today, but I wanted to get this out at least first.

CHANGELOG

  • Added frost-fixed-table component which allows for frozen left and right columns and a fixed header row, but relies on fixed column widths/heights to work properly.
  • Updated ember-frost-core
  • Updated existing components to derive from frost-component now.

})

it('should have real tests', function () {
expect(true).to.equal(false)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops, looks like I forgot to add real tests here. I'll get on that.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling e338ef9 on job13er:add-fixed-table into ** on ciena-frost:master**.

@coveralls
Copy link

coveralls commented Dec 14, 2016

Coverage Status

Changes Unknown when pulling e338ef9 on job13er:add-fixed-table into ** on ciena-frost:master**.

@TheOtherDude
Copy link

LGTM!

@job13er
Copy link
Contributor Author

job13er commented Dec 15, 2016

Approved

Approved with PullApprove

@job13er job13er merged commit 9bd9c63 into ciena-frost:master Dec 15, 2016
@job13er job13er deleted the add-fixed-table branch December 15, 2016 10:55
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.

None yet

3 participants