madcapnmckay / Knockout-UI

A home for rich UI components based on KnockoutJS
265 stars 51 forks source link

Broken with Knockout 2.1.0 #6

Open ZiadJ opened 12 years ago

ZiadJ commented 12 years ago

I tried the dropdown control with the latest version of Knockout it did not seem to like it. Thats a pity because that's about the only UI plugin I know that works with Knockout and looks fantastic.

farina commented 12 years ago

Ya, I really like the structure behind this project...but it looks like it's dead :(. I might try to fork it and go off with my own set.

bsommardahl commented 11 years ago

What would be involved in bringing this project back to life? My team might be interested in taking it on, depending on the level of effort to get it compatible with ko 2.1.0.

madcapnmckay commented 11 years ago

Hi All,

I've been meaning to get back to this project and complete a rewrite and get everything working with 2.1 but have been super busy with new job etc.

I actually more or less completed a rewritten version of the tabs component which I was using as a starter for the other components. I'll try and devote most of this weekend to it and see what I can get done.

I'll aim to get the new version out in the next two weekends.

The new version includes

That being said. I would also love some help maintaining the project in the future so I'd be happy to add anyone that would like to contribute.

Thanks,

Ian

farina commented 11 years ago

I'm kind of in the same boat with the job thing, but I'd love to help out. I'll see what I can do to get involved. It's a great project with some really interesting control concepts.

On Aug 8, 2012, at 10:54 AM, Ian Mckay notifications@github.com wrote:

Hi All,

I've been meaning to get back to this project and complete a rewrite and get everything working with 2.1 but have been super busy with new job etc.

I actually more or less completed a rewritten version of the tabs component which I was using as a starter for the other components. I'll try and devote most of this weekend to it and see what I can get done.

I'll aim to get the new version out in the next two weekends.

The new version includes

A string based template source that allows overriding the templates (wanted this for a long time). Much cleaner code with less configuration UMD pattern similar to KO itself so it should work with AMD, CommonJS etc That being said. I would also love some help maintaining the project in the future so I'd be happy to add anyone that would like to contribute.

Thanks,

Ian

— Reply to this email directly or view it on GitHub.

bsommardahl commented 11 years ago

My team might be interested in helping out. Our company builds knockout apps on a daily basis and we see a lot of potential in your lib. If I could pair program with you for a couple hours on Saturday, I could learn your and decide if we can join in. thoughts? On Aug 8, 2012 11:54 AM, "Ian Mckay" notifications@github.com wrote:

Hi All,

I've been meaning to get back to this project and complete a rewrite and get everything working with 2.1 but have been super busy with new job etc.

I actually more or less completed a rewritten version of the tabs component which I was using as a starter for the other components. I'll try and devote most of this weekend to it and see what I can get done.

I'll aim to get the new version out in the next two weekends.

The new version includes

  • A string based template source that allows overriding the templates (wanted this for a long time).
  • Much cleaner code with less configuration
  • UMD pattern similar to KO itself so it should work with AMD, CommonJS etc

That being said. I would also love some help maintaining the project in the future so I'd be happy to add anyone that would like to contribute.

Thanks,

Ian

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7591628.

farina commented 11 years ago

I'm not even sure how these open source projects work :P. I may not be able to pair program this weekend, but I am interested in helping if at all possible.

I've got a combobox control that I've been working on at the office which might fit right into the KOUI offerings.

Keep me in the loop and I'll see what I can do.

FYI, I'm in Phoenix, AZ (which has it's own timezone essentially). Also, I run a small Mumble server that's able to host a few people for a private voice chat if needed.

I'm also getting pretty experienced with NuGet packages should we decide to package this up into NuGet. I do a lot of UI design and I have all of the graphic design tools we could need.

Farina

On Wed, Aug 8, 2012 at 12:31 PM, Byron Sommardahl notifications@github.comwrote:

My team might be interested in helping out. Our company builds knockout apps on a daily basis and we see a lot of potential in your lib. If I could pair program with you for a couple hours on Saturday, I could learn your and decide if we can join in. thoughts? On Aug 8, 2012 11:54 AM, "Ian Mckay" notifications@github.com wrote:

Hi All,

I've been meaning to get back to this project and complete a rewrite and get everything working with 2.1 but have been super busy with new job etc.

I actually more or less completed a rewritten version of the tabs component which I was using as a starter for the other components. I'll try and devote most of this weekend to it and see what I can get done.

I'll aim to get the new version out in the next two weekends.

The new version includes

  • A string based template source that allows overriding the templates (wanted this for a long time).
  • Much cleaner code with less configuration
  • UMD pattern similar to KO itself so it should work with AMD, CommonJS etc

That being said. I would also love some help maintaining the project in the future so I'd be happy to add anyone that would like to contribute.

Thanks,

Ian

— Reply to this email directly or view it on GitHub< https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7591628>.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7594658.

madcapnmckay commented 11 years ago

@bsommardahl - I'd love to get your input. I'd prefer to do it a week this Saturday, just because I can guarantee my schedule this weekend and I should also have reworked a number of the components by then. Other than the obvious 2.1 compatibility were there any other features, issues you had experienced?

I've been putting off, incorrectly, minor maintenance because of my desire to rewrite the code. I've now become a 100% js engineer for the last 12 months so I know I can improve what was a decent but not spectacular code base. Where necessary I'm aiming to add jasmine tests, proper modularization and full jshint passing code.

I was thinking after I've pushed the new version I can add you all as contributors so you can accept pull requests. Having not collaborated on github I'm not sure what the best way to expose my dev branch to you guys earlier, any ideas? If I can do that you can get hold and critic the code before the big push.

This will be a breaking change so the old code will be grandfathered into a sub directory. I'm aiming to eliminate the legacy jquery template lib and as much of jqueryui as possible. I'd also like to touch optimize the controls but I think that will wait till after next push.

bsommardahl commented 11 years ago

Any chance you can push your progress to github as you are working on it? It would be good to get a chance to try things out as you're developing them. I also ask because I have an immediate need for the tree piece that can't really wait two weeks. I'll either take advantage of your already-done-code or I'll have to roll my own.

On Wed, Aug 8, 2012 at 7:22 PM, Ian Mckay notifications@github.com wrote:

@bsommardahl https://github.com/bsommardahl - I'd love to get your input. I'd prefer to do it a week this Saturday, just because I can guarantee my schedule this weekend and I should also have reworked a number of the components by then. Other than the obvious 2.1 compatibility were there any other features, issues you had experienced?

I've been putting off, incorrectly, minor maintenance because of my desire to rewrite the code. I've now become a 100% js engineer for the last 12 months so I know I can improve what was a decent but not spectacular code base. Where necessary I'm aiming to add jasmine tests, proper modularization and full jshint passing code.

I was thinking after I've pushed the new version I can add you all as contributors so you can accept pull requests. Having not collaborated on github I'm not sure what the best way to expose my dev branch to you guys earlier, any ideas? If I can do that you can get hold and critic the code before the big push.

This will be a breaking change so the old code will be grandfathered into a sub directory. I'm aiming to eliminate the legacy jquery template lib and as much of jqueryui as possible. I'd also like to touch optimize the controls but I think that will wait till after next push.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7602862.

madcapnmckay commented 11 years ago

I would like to avoid pushing incomplete versions especially since its would involve reorganizing and breaking existing components.

I should be able to fix up the tree to work with 2.1. Shouldn't be too much wrong with it. What errors are you getting?

Do you know of any best practice for collaborating on github for major rewrites? Locally I can have a dev branch etc. Would it make sense to fork my own repo as V2. Work on that then merge back into the original and delete the fork?

bsommardahl commented 11 years ago

I would definitely check in your branch so that the branch is visible on github. Since it's a branch, it doesn't matter if it is unfinished.

I forget the errors we were getting. Part of them had to do with jquery.temp.

On 08/08/2012, at 20:10, Ian Mckay notifications@github.com wrote:

I would like to avoid pushing incomplete versions especially since its would involve reorganizing and breaking existing components.

I should be able to fix up the tree to work with 2.1. Shouldn't be too much wrong with it. What errors are you getting?

Do you know of any best practice for collaborating on github for major rewrites? Locally I can have a dev branch etc. Would it make sense to fork my own repo as V2. Work on that then merge back into the original and delete the fork?

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7603485.

madcapnmckay commented 11 years ago

OK great. I'll take a look tonight. If I recall KO 2.1 requires a certain version of jquery.tmpl. Can you make sure it's not simply that the library is using an older version of the tmpl library.

On Thu, Aug 9, 2012 at 3:28 PM, Byron Sommardahl notifications@github.comwrote:

I would definitely check in your branch so that the branch is visible on github. Since it's a branch, it doesn't matter if it is unfinished.

I forget the errors we were getting. Part of them had to do with jquery.temp.

On 08/08/2012, at 20:10, Ian Mckay notifications@github.com wrote:

I would like to avoid pushing incomplete versions especially since its would involve reorganizing and breaking existing components.

I should be able to fix up the tree to work with 2.1. Shouldn't be too much wrong with it. What errors are you getting?

Do you know of any best practice for collaborating on github for major rewrites? Locally I can have a dev branch etc. Would it make sense to fork my own repo as V2. Work on that then merge back into the original and delete the fork?

— Reply to this email directly or view it on GitHub< https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7603485>.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7628994.

farina commented 11 years ago

Are you still planning to use jQuery tmpl for KOUI? I'd just assume pull it out and use KO's native engine. I haven't had any issue using that markup with my applications.

Farina

On Thu, Aug 9, 2012 at 3:35 PM, Ian Mckay notifications@github.com wrote:

OK great. I'll take a look tonight. If I recall KO 2.1 requires a certain version of jquery.tmpl. Can you make sure it's not simply that the library is using an older version of the tmpl library.

On Thu, Aug 9, 2012 at 3:28 PM, Byron Sommardahl notifications@github.comwrote:

I would definitely check in your branch so that the branch is visible on github. Since it's a branch, it doesn't matter if it is unfinished.

I forget the errors we were getting. Part of them had to do with jquery.temp.

On 08/08/2012, at 20:10, Ian Mckay notifications@github.com wrote:

I would like to avoid pushing incomplete versions especially since its would involve reorganizing and breaking existing components.

I should be able to fix up the tree to work with 2.1. Shouldn't be too much wrong with it. What errors are you getting?

Do you know of any best practice for collaborating on github for major rewrites? Locally I can have a dev branch etc. Would it make sense to fork my own repo as V2. Work on that then merge back into the original and delete the fork?

— Reply to this email directly or view it on GitHub<

https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7603485>.

— Reply to this email directly or view it on GitHub< https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7628994>.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7629155.

madcapnmckay commented 11 years ago

I'm definitely planning on removing it in the coming version. Byron needs to get the tree working quickly prior to the next version.

On Thu, Aug 9, 2012 at 4:29 PM, Mark Farina, Jr. notifications@github.comwrote:

Are you still planning to use jQuery tmpl for KOUI? I'd just assume pull it out and use KO's native engine. I haven't had any issue using that markup with my applications.

Farina

On Thu, Aug 9, 2012 at 3:35 PM, Ian Mckay notifications@github.com wrote:

OK great. I'll take a look tonight. If I recall KO 2.1 requires a certain version of jquery.tmpl. Can you make sure it's not simply that the library is using an older version of the tmpl library.

On Thu, Aug 9, 2012 at 3:28 PM, Byron Sommardahl notifications@github.comwrote:

I would definitely check in your branch so that the branch is visible on github. Since it's a branch, it doesn't matter if it is unfinished.

I forget the errors we were getting. Part of them had to do with jquery.temp.

On 08/08/2012, at 20:10, Ian Mckay notifications@github.com wrote:

I would like to avoid pushing incomplete versions especially since its would involve reorganizing and breaking existing components.

I should be able to fix up the tree to work with 2.1. Shouldn't be too much wrong with it. What errors are you getting?

Do you know of any best practice for collaborating on github for major rewrites? Locally I can have a dev branch etc. Would it make sense to fork my own repo as V2. Work on that then merge back into the original and delete the fork?

— Reply to this email directly or view it on GitHub<

https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7603485>.

— Reply to this email directly or view it on GitHub<

https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7628994>.

— Reply to this email directly or view it on GitHub< https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7629155>.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7630174.

bsommardahl commented 11 years ago

We use KO's native templates.

On Thu, Aug 9, 2012 at 5:29 PM, Mark Farina, Jr. notifications@github.comwrote:

Are you still planning to use jQuery tmpl for KOUI? I'd just assume pull it out and use KO's native engine. I haven't had any issue using that markup with my applications.

Farina

On Thu, Aug 9, 2012 at 3:35 PM, Ian Mckay notifications@github.com wrote:

OK great. I'll take a look tonight. If I recall KO 2.1 requires a certain version of jquery.tmpl. Can you make sure it's not simply that the library is using an older version of the tmpl library.

On Thu, Aug 9, 2012 at 3:28 PM, Byron Sommardahl notifications@github.comwrote:

I would definitely check in your branch so that the branch is visible on github. Since it's a branch, it doesn't matter if it is unfinished.

I forget the errors we were getting. Part of them had to do with jquery.temp.

On 08/08/2012, at 20:10, Ian Mckay notifications@github.com wrote:

I would like to avoid pushing incomplete versions especially since its would involve reorganizing and breaking existing components.

I should be able to fix up the tree to work with 2.1. Shouldn't be too much wrong with it. What errors are you getting?

Do you know of any best practice for collaborating on github for major rewrites? Locally I can have a dev branch etc. Would it make sense to fork my own repo as V2. Work on that then merge back into the original and delete the fork?

— Reply to this email directly or view it on GitHub<

https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7603485>.

— Reply to this email directly or view it on GitHub<

https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7628994>.

— Reply to this email directly or view it on GitHub< https://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7629155>.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7630174.

madcapnmckay commented 11 years ago

That's the issue i'm afraid. The current tree won't work without jquery tmpl, it was written almost a year before native templates came out.

I got a prototype rewritten tree completed this weekend. The tree if by far the most complicated component so the others should follow along nicely. It's currently 50% of the size of the original :) Drag and drop still to redo.

bsommardahl commented 11 years ago

Very cool. We went ahead and implemented something quick and ugly to show our client. It should be a pretty big improvement when we implement the new tree.

On Mon, Aug 13, 2012 at 3:24 PM, Ian Mckay notifications@github.com wrote:

That's the issue i'm afraid. The current tree won't work without jquery tmpl, it was written almost a year before native templates came out.

I got a prototype rewritten tree completed this weekend. The tree if by far the most complicated component so the others should follow along nicely. It's currently 50% of the size of the original :) Drag and drop still to redo.

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-7708603.

doctea commented 11 years ago

Any progress on this?

farina commented 11 years ago

I'll be honest, I've done absolutely nothing since the last email :(. I've got this baby on the way, and I'm still remodeling her room to be. I've got some real good ideas for KOUI though :P. I'll code some controls once I get a free minute from house remodeling.

Farina

On Tue, Sep 4, 2012 at 3:12 AM, doctea notifications@github.com wrote:

Any progress on this?

— Reply to this email directly or view it on GitHubhttps://github.com/madcapnmckay/Knockout-UI/issues/6#issuecomment-8258210.

madcapnmckay commented 11 years ago

I finished the rewrite for the tree and tabs. I can push what I have to the dev branch today. I just need to get the other controls converted and then get to grips with creating some github pages to show the examples.

It's taking longer than expected as I haven't had the time to work on it at a weekend yet.

ghost commented 11 years ago

Hey @madcapnmckay, did you manage to push the changes for the tree you had worked on?

choccybutton commented 11 years ago

Just stumbled across this. I'd also be interested in having a play with the new 2.x versions if you have them @madcapnmckay

mzruya commented 11 years ago

Hi, would also like to use test the 2.x versions @madcapnmckay ...

Thanks!

ghost commented 11 years ago

Same here. Would be interested in the KO 2.x development version, @madcapnmckay!

Thanks!

morocarlo commented 11 years ago

I'm working on it...

ste93cry commented 11 years ago

I'm also interested in the KO 2.x version, if you can share what you alredy have it would be great as I'm searching a context menu binding