angular2-grid icon indicating copy to clipboard operation
angular2-grid copied to clipboard

Rewrite & Refactor using TDD

Open BTMorton opened this issue 7 years ago • 5 comments

Over the next few weeks/months I'm going to be doing a rewrite of the library using TDD so there are some actual worthwhile tests. I'm also going to refactor a lot of the code to make it easier to add new features, such as having a component version (#86) and dragging between multiple grids.

While I will try to keep the current functionality the same, there will be some changes, such as moving to a new cascade algorithm similar to how the limit_to_screen works (#87). I will try to keep this issue updated with my progress, any design decisions, and new functionality.

At the end of this rework, there will be a major change update due to the scope of the work involved and the likelihood of there being a breaking change.

BTMorton avatar Mar 05 '17 14:03 BTMorton

Can you provide a little description of how your grid algorithm work (In comments in the code is enough) ? I think that can be useful for new contributors.

NotBad4U avatar Mar 05 '17 14:03 NotBad4U

I will try to, yeah. I'm not the best when it comes to writing comments so don't expect miracles :P, but I'll try to have something written up, and make it more contributor friendly in general.

BTMorton avatar Mar 05 '17 14:03 BTMorton

Currently using this in production, is there an IPR branch I could fork to help? Mostly interested in getting (#87) working, as well as moving to angular 4+.

gordon-to avatar May 02 '17 12:05 gordon-to

@xMrWhite I haven't started yet, unfortunately. Feel free to create a branch to add it in to the current implementation, though.

BTMorton avatar May 05 '17 23:05 BTMorton

Will you possibly be looking at boxes resizing when you drop another box on top of it? #87, will this look to fill the entire grid, or will it stick to the box sizes you've set?

haakotsm avatar May 26 '17 13:05 haakotsm