Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

reliability link #567

Merged
merged 2 commits into from
Nov 23, 2015
Merged

reliability link #567

merged 2 commits into from
Nov 23, 2015

Conversation

tladendo
Copy link
Contributor

@vikasrohit
Copy link
Contributor

lets build

@@ -9,7 +9,7 @@
.subtrack-stats
responsive-carousel(data="vm.subTrackStats", handle="{{vm.handle}}")
.carousel-elem
.flex-wrapper
a.flex-wrapper(ng-class="{'link': item.link, 'no-link': !item.link}", ng-href="{{item.link}}")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it be easier to add this anchor tag with ng-if property?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the tag should be there no matter what, the question is whether it's clickable

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that that anchor tag should be there regardless, but I think <a> is still clickable, even if the cursor isn't a pointer.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i had a problem with clicking these tags at first, but now i'm able to disable clickability.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh ok cool, how did you disable it? pointer-events?

tladendo added a commit that referenced this pull request Nov 23, 2015
@tladendo tladendo merged commit b201b65 into dev Nov 23, 2015
@nlitwin nlitwin deleted the feautre/SUP-1912 branch February 2, 2016 23:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants