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

Extend the edit feature to support dropdowns #1570

Closed
wants to merge 1 commit into from

Conversation

PaulL1
Copy link
Contributor

@PaulL1 PaulL1 commented Sep 22, 2014

Includes creation of a new tutorial, extension of the old tutorial,
and provision of unit tests.

Updated from #1561 to provide for a filter on the options list of the dropdown, allowing translations on the dropdown values (using a tool like angular-translate).

Includes creation of a new tutorial, extension of the old tutorial,
and provision of unit tests.
@swalters
Copy link
Contributor

instead of having Yet Another Option, I prefer the developer to set the editableCellTempate to dropDownEditor.html. This keeps things simple and gives each template a single responsibility without needless ng-if's. We can't keep growing this edit template to include every possible editor.

It's ok to keep these:
editDropdownOptionsArray
editDropdownIdLabel
editDropdownValueLabel
editDropdownFilter
as I can't see a better way to do it

@PaulL1
Copy link
Contributor Author

PaulL1 commented Sep 23, 2014

Replaced by #1579

@PaulL1 PaulL1 closed this Sep 23, 2014
@c0bra c0bra removed the in progress label Sep 23, 2014
@PaulL1 PaulL1 deleted the edit_dropdown_updated branch September 26, 2014 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants