Skip to content
This repository has been archived by the owner. It is now read-only.

Update rating.html #3048

Closed
wants to merge 1 commit into from
Closed

Update rating.html #3048

wants to merge 1 commit into from

Conversation

@jr0jas
Copy link

jr0jas commented Dec 4, 2014

Avoid the stop propagation if the parent of the Rating have click event

Avoid the stop propagation if the parent of the Rating have click event
@wesleycho
Copy link
Member

wesleycho commented Mar 16, 2015

This should be something configurable, via attribute such as rating-config, with it being an object with a stopPropagation key that can be set to true or false. Default behavior should be true.

This will require unit tests as well.

@wesleycho wesleycho force-pushed the angular-ui:master branch from e373941 to 2c2dba6 Mar 23, 2015
@wesleycho
Copy link
Member

wesleycho commented Jul 30, 2015

Closing due to flawed implementation - feel free to open a new PR that addresses the concerns mentioned here.

@wesleycho wesleycho closed this Jul 30, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.