search

gtg111a / campusmack

My First App
1 stars 0 forks source link

Add Recipients #2

Closed arpadlukacs closed 13 years ago

arpadlukacs commented 13 years ago

Fixing contacts list so it doesn't collapse when you click on one of the groups

gtg111a commented 13 years ago

Tested the "Add Recipients" list by clicking on a group and the list still collapses.

arpadlukacs commented 13 years ago

Yes, that was intended. This is how you can switch group. I only fixed the function behind the checkbox. Now, it does not switch group when you click on a checkbox. It seems I misunderstood what you wanted. If I remove the ability to switch group, users will not be able to add one contact from a group, they can only add all or none. Is this what you want? If you can add a group's contacts by clicking on it's name, why we need checkboxes? Should I remove the functionality from the names of groups completely? Replace them with simple text, so only the checkbox would work?

gtg111a commented 13 years ago

I just did the checkbox and it collapsed. You understood me correctly, it's just not working the way we want it yet.

-Mike

On Wed, Oct 19, 2011 at 6:08 AM, arpadlukacs < reply@reply.github.com>wrote:

Yes, that was intended. This is how you can switch group. I only fixed the function behind the checkbox. Now, it does not switch group when you click on a checkbox. It seems I misunderstood what you wanted. If I remove the ability to switch group, users will not be able to add one contact from a group, they can only add all or none. Is this what you want? If you can add a group's contacts by clicking on it's name, why we need checkboxes? Should I remove the functionality from the names of groups completely? Replace them with simple text, so only the checkbox would work?

Reply to this email directly or view it on GitHub: https://github.com/gtg111a/campusmack/issues/2#issuecomment-2454038

arpadlukacs commented 13 years ago

So, it works in Chrome, IE8 and Safari, but not in Firefox.