-
Notifications
You must be signed in to change notification settings - Fork 0
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 implementation #1
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to add doc and examples.
Need to add angular module that can be added to the main project. |
example/js/controller.js
Outdated
window.scope = $scope; | ||
window.rootScope = $rootScope; | ||
|
||
console.log(data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trean remove please console log
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
example/js/scalableTable.js
Outdated
@@ -0,0 +1,840 @@ | |||
angular.module('scalable-table', []) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trean why don't you use main scalableTable.js directive instead of duplicating it here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@myfoxtail I thought that it will be easier to an user to run the example. It don't requires from an user path to URl which looks like http://some-url/example. But other side it can confuse user. So maybe it needs to changes. Is it OK if I'll remove this file from example directory and use main directive?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trean The problem is that if you change something in the main script you have to change it in the example. It's not convenient. I think yes, you can use the file from the main directive.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@myfoxtail I have used main file now. But I should been change the line 83 to scope.$on('$destroy', function() {
There is removed $
, because it caused an error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trean ok, thanks, we will handle it later
example/js/controller.js
Outdated
return arr; | ||
}); | ||
console.log($scope.tableData); | ||
console.log($scope.tableData.length); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
example/js/controller.js
Outdated
}; | ||
|
||
$scope.gotoFoo = function (key) { | ||
console.log(key) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
example/js/scalableTable.js
Outdated
|
||
return { | ||
post: function(scope, elem, attrs) { | ||
console.log(scope.scalableTableColumns); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trean remove please console.log here too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It do not need anymore (I use main file now where is not exist this line).
scalableTable.js
Outdated
|
||
return { | ||
post: function(scope, elem, attrs) { | ||
console.log(scope.scalableTableColumns); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trean remove console log here too, please.
@trean could you also please add working example that includes 10000 rows and 300 columns for example. |
|
Need to add doc and examples