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 implementation #1

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

add implementation #1

wants to merge 7 commits into from

Conversation

okneigres
Copy link

@okneigres okneigres commented Dec 15, 2016

Need to add doc and examples

Copy link
Author

@okneigres okneigres left a 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.

@myfoxtail
Copy link
Member

Need to add angular module that can be added to the main project.

@myfoxtail myfoxtail added this to the first version release milestone Dec 18, 2016
window.scope = $scope;
window.rootScope = $rootScope;

console.log(data);
Copy link
Member

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

Copy link

Choose a reason for hiding this comment

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

done

@@ -0,0 +1,840 @@
angular.module('scalable-table', [])
Copy link
Member

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?

Copy link

@trean trean Feb 20, 2017

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?

Copy link
Member

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.

Copy link

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.

Copy link
Member

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

return arr;
});
console.log($scope.tableData);
console.log($scope.tableData.length);
Copy link
Member

Choose a reason for hiding this comment

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

and here...

Copy link

Choose a reason for hiding this comment

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

done

};

$scope.gotoFoo = function (key) {
console.log(key)
Copy link
Member

Choose a reason for hiding this comment

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

and here...

Copy link

Choose a reason for hiding this comment

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

done


return {
post: function(scope, elem, attrs) {
console.log(scope.scalableTableColumns);
Copy link
Member

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.

Copy link

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);
Copy link
Member

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.

@myfoxtail
Copy link
Member

@trean could you also please add working example that includes 10000 rows and 300 columns for example.

@myfoxtail
Copy link
Member

image
@trean please make some changes in UI so it looks cool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants